-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: ui issues in compose #970
Conversation
Bug Fixes
Features
StylesChore
ContributorsCommit-Lint commandsYou can trigger Commit-Lint actions by commenting on this PR:
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #970 +/- ##
=======================================
Coverage 77.49% 77.50%
=======================================
Files 150 150
Lines 2173 2178 +5
Branches 233 236 +3
=======================================
+ Hits 1684 1688 +4
- Misses 412 413 +1
Partials 77 77
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Remove the gap on the sides in the config screen when selecting an item |
Animation going into a series detail screen? |
Rating dialogue needs its buttons moved to the side to match other dialogs |
In the settings screen the clickable area is too small, fix the order of modifiers to fix this.
If we get a 401 during normal operation, then attempt to refresh and throw an error if we cant.
Add a new extension method to allow autofill and use this on the credentials screen.
Update padding to include the whole line in the config screen.
Move the dialog buttons to be end aligned like the other dialogs.
648f1c8
to
163e2f3
Compare
Fix various ui issues that arose from switching to compose.
Except 1410f07, this is something that has always been around that needed fixing.