Skip to content

Add reset button to navbar corner radius slider#160

Merged
theovilardo merged 1 commit intotheovilardo:masterfrom
ColbyCabrera:feature/add-reset-button-navbar-corner-radius
Sep 21, 2025
Merged

Add reset button to navbar corner radius slider#160
theovilardo merged 1 commit intotheovilardo:masterfrom
ColbyCabrera:feature/add-reset-button-navbar-corner-radius

Conversation

@ColbyCabrera
Copy link
Copy Markdown
Contributor

This pull request updates the NavBarCornerRadiusScreen to improve usability by introducing a reset button for the navigation bar corner radius, refining the UI, and enhancing state management. The most significant changes are grouped below:

New Feature: Reset Button

  • Added a "Reset" button that appears when the user adjusts the slider from its default value. Clicking it resets the slider to the default corner radius (DEFAULT_NAV_BAR_CORNER_RADIUS) and updates the UI accordingly. [1] [2]

UI/UX Improvements

  • Changed the instructional text to clarify that the user should match the navbar shape's corners with their device's corners, improving clarity.
  • Updated the layout of the top app bar and action buttons for better alignment and spacing, replacing the TextButton with a Button and using a Row for button arrangement.

State Management Enhancements

  • Introduced hasBeenAdjusted state to track if the slider value has deviated from the default, controlling the visibility of the reset button. [1] [2]
  • Switched from mutableStateOf to mutableFloatStateOf to fix warning,
Screenshot 2025-09-15 234333

@Ayaanh001
Copy link
Copy Markdown
Contributor

Nice work! I recently raised this issues #148
And u have fixed it. Can u also trying adding a Folders section along with Tracks, Playlist, Albums, etc in Library Tab cuz it's difficult to find or check the songs in or from a particular Directory. Like samsung music shows, Here it is for your reference.
484013994-7465de48-ae77-4697-958d-a92b5b6b9899
484013995-660ea1d5-da0b-4bb0-9afb-18648d86562b

In Library Tab
Screenshot_2025-09-18-03-28-44-25_245b0f76029d0720fff82e331ce82234

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