Skip to content
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

Modify SwitchAccount button so it doesn't overlap with Search menu in RootFilesFragment #1237

Merged
merged 3 commits into from
Mar 22, 2024

Conversation

LunarX
Copy link
Contributor

@LunarX LunarX commented Mar 4, 2024

The text below the transparent button needs to still be displayed the exact same way, so we overlay a transparent button to reduce its size independently. By limiting its size we prevent wide switch drive buttons from overlapping with the search menu from the toolbar in RootFilesFragment blocking some clicks that were aimed at the search menu button

@LunarX LunarX self-assigned this Mar 4, 2024
@LunarX LunarX requested a review from a team as a code owner March 4, 2024 12:31
@LunarX LunarX force-pushed the overlay-transparent-button branch from 0d5967f to 6c7649d Compare March 5, 2024 12:21
@KevinBoulongne KevinBoulongne force-pushed the overlay-transparent-button branch from 6c7649d to 04a9b92 Compare March 6, 2024 07:08
@LunarX LunarX force-pushed the overlay-transparent-button branch from 05990c0 to 9a357a6 Compare March 13, 2024 08:21
@KevinBoulongne KevinBoulongne force-pushed the overlay-transparent-button branch from 9a357a6 to 9153dc5 Compare March 18, 2024 06:13
LunarX and others added 2 commits March 18, 2024 07:17
…n RootFilesFragment

The text below the transparent button needs to still be displayed the exact same way, so we overlay a transparent button to reduce its size independently. By limiting its size we prevent wide switch drive buttons from overlapping with the search menu from the toolbar in RootFilesFragment blocking some clicks that were aimed at the search menu button
@KevinBoulongne KevinBoulongne force-pushed the overlay-transparent-button branch from 9153dc5 to 19a2db0 Compare March 18, 2024 06:17
@KevinBoulongne KevinBoulongne changed the title Modify switch account button so it doesn't overlap with search menu in RootFilesFragment Modify SwitchAccount button so it doesn't overlap with Search menu in RootFilesFragment Mar 18, 2024
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@tevincent tevincent merged commit 66679fa into apiv3-ui Mar 22, 2024
4 checks passed
@tevincent tevincent deleted the overlay-transparent-button branch March 22, 2024 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants