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

feat(announcements): banner and placement #766

Merged
merged 5 commits into from
Feb 29, 2024

Conversation

GeopJr
Copy link
Owner

@GeopJr GeopJr commented Feb 3, 2024

This PR does the following

  • Move announcements from the sidebar to the menu
  • Show a banner when new announcements are available

The questions are:

  • Should it even move to the menu? The idea was that it takes space in the sidebar for almost no reason - unless there are announcements it's probably useless 99% of the time. However the workflow of showing it when there are new announcements (with a badge like notifications) and hiding it when there are not, is not ideal because it will either be visible until the request to announcements completes and then disappear if there are none OR be hidden and appear if there are any. Both of these have a UI shift
  • Should by clicking "View" also mark all announcements as read? I think so, but you can mark as read individual announcements too so I'm not sure what's more desirable
Screencast.from.2024-02-03.19-22-11.webm

@LukaszH77
Copy link
Contributor

The idea was that it takes space in the sidebar for almost no reason - unless there are announcements it's probably useless 99% of the time.

Yep, pretty much. Is there a way (and is it HIG-compliant if so) to have announcement in the sidebar only when there are unread announcements? But then again, there would be no way to see them after marking as viewed...

@GeopJr
Copy link
Owner Author

GeopJr commented Feb 3, 2024

Not visible in the recording above but it was moved to the main menu (the one next to "Tuba"), and will be there whether or not there are unread announcements

Is there a way (and is it HIG-compliant if so) to have announcement in the sidebar only when there are unread announcements?

Tobias also suggested that in the circle review but my concerns with that is the UI shift. I tried to explain why on the first point in the issue but it's not that clear:

Basically, a request needs to be done to get the announcements to see if there are any unread ones. While that is being done, we need to decide whether "Announcements" should be in the sidebar until then and disappear after the request is done and there are no announcements OR, the opposite, have it not be there initially and only appear in the sidebar if there are any announcements

The problem with both is that the sidebar height will change from the initial state to the one after the request is done. If a user is trying to click "Follow Requests" right when the announcements request finishes, they'll click "Announcements" if it appears then

(On that note, "Follow Requests" needs to move away too, since it's only useful when there are any follow requests". Another banner?)

there should really be a dismiss all endpoit :/ hopefully doesn't spam any instances
@GeopJr GeopJr marked this pull request as ready for review February 29, 2024 19:36
@GeopJr GeopJr merged commit cf9ca6d into main Feb 29, 2024
6 checks passed
@GeopJr GeopJr deleted the feat/announcements/banner-and-placement branch February 29, 2024 19:50
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.

None yet

2 participants