סגנונות חדשים לתפריט נפתח בכותרת / תפריט התראות

This is merged → UX: Notification panel update by jordanvidrine · Pull Request #33795 · discourse/discourse · GitHub


UPDATE:

This is now active on meta for all users

Summary

Experiment for a new look & style for header dropdown menus along with notification item styles.

We remove the “active” border styling from dropdowns in header. This border adds a complication to styling when creating themes.

We add spacing, and change the “active” icon styling on the notification menu.

Screenshots

16 לייקים

Cool, this experiment is exciting! Love the new look here Jordan :smiley:

Some initial feedback: the spacing feels a bit tight, could we experiment with a tad bit more spacing? I think it’s because of the padding on the ✓ Dismiss buttons is quite larger. I think it would be nice if we aligned them with the same padding (0.75rem ?)

Something like this?:

9 לייקים

I definitely miss the titles of the topics. When there is one like or two likes by users with short names, there is enough information. But one or two words of the title are not enough for me.

10 לייקים

Thanks for the feedback you two. Both are really spot on :+1: Im going to let more feedback pile up before addressing, but rest assured, I will be adding these to the list.

7 לייקים

כמה רשמים ראשונים:

עבור ברירת מחדל והוריזון:

  • בדומה לקיגן, התחושה שלי היא שהריווח קצת צפוף מדי
  • העובדה שאין הצללה בחלל השלילי בין הפריטים גורמת לי להרגיש עמוס יותר, בהשוואה למצב שבו הכל השתלב יחד

ספציפי להוריזון:

  • בהוריזון, לרקע של הפריטים יש צבע - האם זה אמור להיות כך? (בברירת מחדל, הוא עכשיו אפור).
  • בהוריזון, הפינות המעוגלות בצד שמאל למעלה נראות לי לא טוב… המכל נראה חד יותר מהפריטים שבתוכו - בברירת מחדל, זה מרגיש שהם מתאימים יותר.
5 לייקים

מצאתי שהפשרה של יצירת יותר מקום להתראות ישנות (שאולי כבר טיפלתי בהן) בתמורה לאיבוד ההקשר של ההתראות החדשות יותר (שאני עדיין צריך לטפל בהן): פשרה לא צפויה.

זה מתקשר לאותו הדבר ש-“@moin” אמר למעלה, והעיצוב הצפוף יותר, כפי שציין “@keegan”, עומד בסתירה למעבר לעבר פחות צפיפות שאליו כיוונו עם רכיבים כמו סרגל הצד.

6 לייקים

This is due to the larger border radius set by horizon && core setting this radius as “large”. I have since moved this border in core to inherit the regularly set border radius.

Good callout. This is due to an incorrect setting I think.

3 לייקים

wrt spacing… what do you think of this?

Here I am limiting the title to one line, but allowing the signifier to have it’s own line as well. I’ve also added left and right padding of the same space to match the buttons below.

7 לייקים

This is possibly deviating a bit too much from the purpose of the experiment, but I think a larger issue that restricts our design is the amount of notifications we show. I don’t think we need to show a whole page long of notifications just because the browser height permits it. Users can click show more and be directed to /my/notifications when they need to see everything.

:100: I think this is a step in the right direction, and we could possibly:

  • reduce the number of notifications we show
  • show more information per notification

This design I found on Dribbble for example, has a nice ratio of amount of notifications / information on each notification / density.

10 לייקים

אה כן, אני ממש אוהב את התצוגה הזו.

3 לייקים

I also noticed that the avatars aren’t as covered by the icon in your screenshot as in mine. Choosing “smaller” for the text size in the preferences seems to cause that. Maybe it was the case before, too, that a larger part of the avatar was covered because I changed the text size, but now the avatars are more covered than before. (I still prefer the old icon-only version, so I don’t mind.)

For comparison: with the experiment and in safe mode

And some icons are indented

3 לייקים

Thoughts?

3 לייקים

I think we need more space around the notifications, this also would fix this misalignment:

image

Can these radii be the same?

image

I don’t feel like the grey background works well to indicate new and active… we usually use tertiary for both new and active states and grey for hover

image

The icons feel a little too small to me now, I find myself squinting to make out some

image

image

Do you have any ideas around how this would work out with search menu changes? we recently went full-width on highlights there so it feels a little off to be back to boxes

7 לייקים

I like the stylechange on the tab in general.

One more spot to have a bit more spacing: the click target for the tabs on mobile is very small

It’s quite hard to tap.

3 לייקים

TBH, I didn’t enjoy it and prefer the old layout.

  • As others said above, the layout is too packed. The screenshot above improves that.
  • The border radius and gap between all items makes the list feels busier. It adds visual noise to the information:

  • The background colors for the unread items don’t look as good in other color palettes. They all look grey-ish and lost the color accent.

Old:

New:

  • Since we don’t have the border anymore separating the notifications from the icons, the misalignment between them and the items gives the impression that something is off, specially in the first icon.

  • The icons are too small, and in mobile mode the spacing is off. It makes it harder to click on the correct icon. I already accidentally clicked a notification instead of the icon, twice.

I disagree. It’s nice that we take advantage of the available viewport.

This would be nice.

7 לייקים

I am merging some changes. Here is the latest iteration.

how would the avatar look aligned to the top so it’s always closer to where the username is? I wonder if the centered avatars would feel a bit “unanchored” if I had a lot of double-line notifications… very rough comparison

לייק 1

ניסיתי את זה באופן מקומי וזה הרגיש “מוזר”, אבל אני יכול ליישם את זה כאן כדי לראות איך זה מרגיש לכולם.

לייק 1

Can you try increasing the gap between the items to match the outer padding to see how it feels?

לייק 1

לייק 1