tanmayrajani / notifications-preview-github

Browser Extension: preview GitHub notifications with same page pop-overs
https://chrome.google.com/webstore/detail/github-notifications-prev/kgilejfahkjidpaclkepbdoeioeohfmj?hl=en&gl=IN
MIT License
141 stars 15 forks source link

Width needs to be made bigger #102

Closed yakov116 closed 3 years ago

yakov116 commented 3 years ago

Since the mark as read has been restored I find myself accidentally clicking the "read" checkbox

silverwind commented 3 years ago

I think it's wide enough already, especially on small screens.

tanmayrajani commented 3 years ago

Yes, I think so too. I mean, it's "compact" mode for a reason. If you want more width, you can always go back to non-compact mode using the options.

Still, if more people think we should increase the width, I'd be happy to.

silverwind commented 3 years ago

What could be done is a minimum or fixed width instead of completely dynamic as it is now. For example currently the hover effect cuts into small items a bit too much:

After adding a minimum width like:

.NPG-dropdown .notifications-list-item {
  min-width: 400px;
}