8000 `unread-anywhere` - Add button by fregante · Pull Request #8638 · refined-github/refined-github · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@fregante
Copy link
Member
@fregante fregante commented Aug 23, 2025

The notification buttons is a rather vital part of GitHub so changes here need to be careful.

I thought about using a larger blue bubble but that is both visually distracting and not really clickable (it's ::before, which you can't detect in a click event)

This style was one of the ones initially suggested.

Also I left it visible even if there are no notifications, because GitHub's UI is not necessarily up to date (#1426)

Screenshot

Screenshot 3 Screenshot 6

@fregante fregante merged commit 559c4e8 into main Aug 23, 2025
9 checks passed
@FloEdelmann FloEdelmann deleted the open-unread branch August 25, 2025 08:37
@FloEdelmann
Copy link
Member
FloEdelmann commented Aug 25, 2025

To me, that style looks like a broken button (or a missing icon). I'll disable the feature anyway (because I have too many unread notifications for it to be useful), so I don't care enough to open an issue about it.

@ewjoachim
Copy link

I don't plan on using this feature, because to me, keeping unread notifications unread is an important part of making sure I read them at the right time, and I have a mix of personal and professional notifications, so I never want to mix them.
I find the position of the button a bit annoying, as a single pixel can be the difference between the button I use 60 times per day, and the button I must never click lest it will change something that cannot be undone (make notifs as read).
(I wonder if there are accessibility guidelines for how small and close together 2 buttons may be before it starts to be really hard to some users?)

There was a suggestion on making it on alt+click that was set aside as "too hard to discover to be useful". What about changing the tooltip of the button to "You have unread notifications. Use alt+click to open them all"

That said, I've disabled the feature. I think it's the first time I had to do that, and I'm perfectly fine doing that, but I wanted to offer some feedback so that there's a chance to revisit design choices. But, again, I'm perfectly fine if nothing changes.

8000
@fregante
Copy link
Member Author
fregante commented Aug 25, 2025

Thanks for the feedback! Keep them coming.

This is the reasoning:

  • the button must exist because I myself forgot I even wrote the feature for several months. Shortcuts are effectively hidden features.
  • the button cannot be too big because it causes content shift on the sidebar (e.g. to try to click the native Notifications link, but suddenly the new button appears)
  • the button must be near the native link, not hidden in menus for example

This is the closest I could get:

  • native style, it only lacks an icon
  • its target is small enough to make the action more difficult
  • the action is non-destructive:
    • up to 10 tabs are opened and in each there's a bar to "mark as unread" again

Improvements are still possible, so I'm open to suggestions. For example:

  • add a confirmation every time (annoying, but can be accepted with enter)
  • limit to 5 tabs
  • change style, but this is the hard part given the limitations

@ewjoachim
Copy link

in each there's a bar to "mark as unread" again

there is ?! I never saw it ! (I'm seeing it now)

@fregante
Copy link
Member Author

heheh yes! For future reference, this is what I see after opening your notification:

Screenshot

@ewjoachim
Copy link
ewjoachim commented Aug 25, 2025

A suggestion: a dropdown in the style of the copilot button ?
Screenshot 2025-08-25 at 19 07 02

It means it takes 2 clicks, BUT they're much closer from one another than a confirm dialog

Edit: an additional thought: it's fairly obvious what each button in the top bar does without needing the tooltip. (Assuming you remember that the circle with dot now means "issue", but that ship has sailed). That small button now really is something you need the tooltip to have an idea what it does. It sure means "something something notifications", but it's hard to guess.

@fregante
Copy link
Member Author

A suggestion: a dropdown in the style of the copilot button ?

This violates the second rule: can't be too big.

I think I'd need to go back to playing with the blue indicator, but without a blue center because it makes it too visually distracting

Screenshot 1
  • small/native enough
  • it doesn't shift content at all

I think there's no way to fulfill these two constraints at the same time:

  • looks 100% native
  • doesn't shift content

This is basically the only option, but it still looks weird to have a "duplicate" notifications action so far away from the notifications.

Screenshot

it's hard to guess.

As you said, the "too many icons" ship has sailed. You will find out what it means by hovering it.

@fregante
Copy link
Member Author

See #8642

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

Development

Successfully merging this pull request may close these issues.

unread-anywhere - Add button

4 participants

0