Skip to content

Fix group padding on the 'Feeds' page#949

Merged
Ashinch merged 1 commit intoReadYouApp:mainfrom
ymcx:fix-group-alignment
Mar 20, 2025
Merged

Fix group padding on the 'Feeds' page#949
Ashinch merged 1 commit intoReadYouApp:mainfrom
ymcx:fix-group-alignment

Conversation

@ymcx
Copy link
Contributor

@ymcx ymcx commented Jan 25, 2025

The group spacing is incorrect on the 'All' tab of the Feeds page. A 16dp spacer is supposed to be added for each visible feed. Yet if the user has no feeds in their Default group and the group is hidden, a spacer is added nonetheless.

I've included some screenshots below. At the top, you can see what the 'Unread' and 'All' pages looked like before. At the bottom is what it is supposed to look like, with the inconsistent padding removed.
Comparison

@Ashinch
Copy link
Member

Ashinch commented Mar 20, 2025

Thanks!

@Ashinch Ashinch merged commit eb3b826 into ReadYouApp:main Mar 20, 2025
2 checks passed
@Ashinch Ashinch added the bug:ui label Mar 20, 2025
@Ashinch Ashinch added this to the 0.12.0 milestone Mar 20, 2025
@ymcx ymcx deleted the fix-group-alignment branch March 22, 2025 18:52
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.

2 participants