fix: Replace Spacer with Expanded to avoid overflow #1791
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Team are going to make a joke now , 44 hours on decoration for a 10 seconds change
馃槉 Issue
@MarcaDian created an issue with our source switcher regarding the
[AlertDialog]
[title]
being too long and caused the[IconButton]
to overflowed out of the view here:The relevant code was line 30-40 in Settings Fragment - Manage Sources
revanced-manager/lib/ui/views/settings/settingsFragment/settings_manage_sources.dart
Lines 30 to 40 in c209c32
[Spacer]
tried to take up all the remaining space after those two widgets (Text & IconButton) by default it will take one yade yada (I hope I'm right) - tl;dr: we basically have very limited space - now obviously as you can see in #1790 that ended up being a failure, so what I do is we switch to[Expanded]
which surprisingly works a lot better... (well yeah no sh-)馃敡 Testing note
Testing was done in "simulated" environment, which means I didn't edit ReVanced Manager code, but I recreated it in some online editor, specifically DartPad which work super well in my opinion.
20240327_123737.mp4
(It's encoded using HEVC, you might need additional setup, deal with it )
fix: #1790