Skip to content

Conversation

@setchy
Copy link
Member

@setchysetchy commented Apr 1, 2024

#750

To do list

  • Fix settings view - new option now causes the settings view to scroll, with the footer overflowing
  • Fix App.test.tsx tests
  • Add new unit tests for coverage

Copy link
Contributor

@adufradufr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested, lgtm 👍 It's indeed much cleaner now

@setchy
Copy link
MemberAuthor

Just need a solution to the Settings UI layout/scrolling then we're good to go

@setchysetchy mentioned this pull request Apr 2, 2024
@setchy
Copy link
MemberAuthor

Just need a solution to the Settings UI layout/scrolling then we're good to go

Reworked the Settings screen within #947

Copy link
Member

@afonsojramosafonsojramos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lovely to see how modular this change became, thanks @setchy and @adufr!

@afonsojramosafonsojramos added this to the Release 5.2.0 milestone Apr 2, 2024
@setchysetchy merged commit 37d3bef into mainApr 2, 2024
@setchysetchy deleted the feature/count-title branch April 2, 2024 16:53
@setchy
Copy link
MemberAuthor

@adufr - shout out, I'm really enjoying this feature. It's nice seeing at a glance just how "green" my notifications backlog is 👏

@adufr
Copy link
Contributor

@adufr - shout out, I'm really enjoying this feature. It's nice seeing at a glance just how "green" my notifications backlog is 👏

Only made possible thanks to you ;)

@bmulholland
Copy link
Collaborator

Yeah actually I was opposed to this at first, but now I use it quite a bit 😅. Thank you!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@setchy@adufr@bmulholland@afonsojramos