# Feedback on new 🍔 and user menus

**URL:** https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519
**Category:** UX
**Created:** [25 augustus 2015 om 22:09 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519 "2015-08-25T22:09:49Z")
**Posts on this page:** 20
**Page:** 10

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [4 september 2015 om 18:25 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/194 "2015-09-04T18:25:48Z")

</div>

The dimmer profile links are intentional since 99% of the time when you go there you are there to read your notifications, _not_ there to visit your profile, bookmarks, direct link to messages outside the context of a notification, or profile settings.

Also @kuro categories is a heading there.

---

<div class="post-metadata">

### Author: ![Kuro](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/kuro/32/38491_2.png) [@Kuro](https://meta.discourse.org/u/Kuro)
#### Post date: [4 september 2015 om 18:50 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/195 "2015-09-04T18:50:22Z")

</div>

> [@codinghorror](#):
>
> Also @kuro categories is a heading there.

Well, yes. I think thats what I wanted to express. Though while it might be intented as a heading it looks like any other link above it, has the same hover effect as any other link above it and actually links as any other link above it.  
What I am trying to get across is that right now it kinda looks like somebody just typed in the hr too early as it fits way better with the stuff above it then the things in its section.  
If you change the style of that heading, I assume it would become clearer.

I am not radically forcing you to change it, though. It is merely a suggestion.

---

<div class="post-metadata">

### Author: ![RaceProUK](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/raceprouk/32/115844_2.png) [@RaceProUK](https://meta.discourse.org/u/RaceProUK)
#### Post date: [4 september 2015 om 18:55 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/196 "2015-09-04T18:55:27Z")

</div>

Maybe, instead of the header link above, an ‘All Categories’ link below?

 ![](https://global.discourse-cdn.com/meta/original/3X/d/a/dae1e2b17926be39f7bdf2dcc6c44b5088a20b78.png)  
I know it’s a horrible hack job; I did it with the Chrome Devtools. But it’s good enough to show the sort of thing I mean.

---

<div class="post-metadata">

### Author: ![loopback0](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/loopback0/32/37486_2.png) [@loopback0](https://meta.discourse.org/u/loopback0)
#### Post date: [4 september 2015 om 19:17 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/197 "2015-09-04T19:17:40Z")

</div>

> [@codinghorror](#):
>
> Also @kuro categories is a heading there.

I’m not sure it’s required. Those are clearly categories and the other sections don’t have headings.

 ![](https://global.discourse-cdn.com/meta/original/3X/d/1/d119705644ff57fdcf0ec1e914da70222014d5bd.png)

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [4 september 2015 om 19:18 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/198 "2015-09-04T19:18:57Z")

</div>

Yes, but how do you get to the /categories page from hamburger now?

---

<div class="post-metadata">

### Author: ![loopback0](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/loopback0/32/37486_2.png) [@loopback0](https://meta.discourse.org/u/loopback0)
#### Post date: [4 september 2015 om 19:20 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/199 "2015-09-04T19:20:06Z")

</div>

Oops.  
Add another entry that’s styled like the rest called “all” or “all categories”?

---

<div class="post-metadata">

### Author: ![RaceProUK](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/raceprouk/32/115844_2.png) [@RaceProUK](https://meta.discourse.org/u/RaceProUK)
#### Post date: [4 september 2015 om 19:23 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/200 "2015-09-04T19:23:47Z")

</div>

Or something like I posted a few back up-topic?

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [4 september 2015 om 19:24 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/201 "2015-09-04T19:24:00Z")

</div>

I think we’re done here.

---

<div class="post-metadata">

### Author: ![eviltrout](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/eviltrout/32/5275_2.png) [@eviltrout](https://meta.discourse.org/u/eviltrout)
#### Post date: [4 september 2015 om 19:37 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/202 "2015-09-04T19:37:56Z")

</div>

No that seems wrong too.

[https://github.com/discourse/discourse/commit/49096e5801ef5b262dfe1c4356f567d575902f9a](https://github.com/discourse/discourse/commit/49096e5801ef5b262dfe1c4356f567d575902f9a)

---

<div class="post-metadata">

### Author: ![JSey](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/jsey/32/3540_2.png) [@JSey](https://meta.discourse.org/u/JSey)
#### Post date: [4 september 2015 om 19:57 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/203 "2015-09-04T19:57:19Z")

</div>

Here’s something a little nasty I found. This surfaced with the new 🍔 (at least, I did not recall seing it earlier):

The style for `.d-header .icons .unread-private-messages` is specified as `left: 82px` - that’s fine as long as there are no other icons in the top right. If you add a plugin that uses the plugin outlet to inject another icon, your notifications start to wander around like this:

![](https://global.discourse-cdn.com/meta/original/3X/d/e/de7198bfd84a308387fa623aaa59691a07b7fdb3.png)

which is highly confusing right now, but will completely break with the next icon. I suggest to drop the `position: absolute` in favor of relative positioning - the badge `<a>` tag is within the `<li>`, so there is no structural problem here.

---

<div class="post-metadata">

### Author: ![eviltrout](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/eviltrout/32/5275_2.png) [@eviltrout](https://meta.discourse.org/u/eviltrout)
#### Post date: [4 september 2015 om 20:12 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/204 "2015-09-04T20:12:38Z")

</div>

Moving the Akismet link was surprisingly tough since it uses an outlet that has been moved. I added a backwards compatibility layer to re-wire outlets, and it seems to be working. Deploying shortly.

---

<div class="post-metadata">

### Author: ![Kuro](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/kuro/32/38491_2.png) [@Kuro](https://meta.discourse.org/u/Kuro)
#### Post date: [4 september 2015 om 22:44 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/205 "2015-09-04T22:44:12Z")

</div>

Hey, I was the one who brought it up and you argued with loopback0 and RaceProUK.  
At least let me pitch something before locking this conversation-path up, please.

See, I am really minimalistic in some ways. And while you can probably style it better, for me a simple  
`text-align:center` on that link makes a huge difference.

 ![](https://global.discourse-cdn.com/meta/original/3X/c/2/c2aa5a205e2916c1ff0e4baebc0fe062abcf086f.png)  
This way - while not being optimal - it at least covers both sides of the 2 col layout. It is also different from the links on top (being left-aligned and in 2 cols as well) and to me it seems like its really easy to implement without any drawbacks, really.

Filed Under: If you don’t like it, feel free to simply dismiss it or tell me why you think this is a bad idea. 😃

---

<div class="post-metadata">

### Author: ![Tom\_Newsom](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/tom_newsom/32/115981_2.png) [@Tom\_Newsom](https://meta.discourse.org/u/Tom_Newsom)
#### Post date: [4 september 2015 om 23:11 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/206 "2015-09-04T23:11:04Z")

</div>

Crazy?

 ![](https://global.discourse-cdn.com/meta/original/3X/a/e/aec41e9a5fce2fe6864f771917b9c6817f4c5c72.png)

---

<div class="post-metadata">

### Author: ![wesochuck](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/wesochuck/32/108706_2.png) [@wesochuck](https://meta.discourse.org/u/wesochuck)
#### Post date: [4 september 2015 om 23:23 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/207 "2015-09-04T23:23:03Z")

</div>

Because you should [rarely center anything](https://books.google.com/books?id=gUWPBQAAQBAJ&pg=PA38&lpg=PA38&dq=non-designer%27s+design+book+never+center&source=bl&ots=yRuIVcqMHs&sig=xxnSMfsOYV76O4q1L4LsE08PMr0&hl=en&sa=X&ved=0CDMQ6AEwA2oVChMIgOPCssLexwIVzokNCh2b-wCB#v=onepage&q=non-designer%27s%20design%20book%20never%20center&f=false) 😉 I’m not sure that there is any other element in Discourse I can think of that is centered.

---

<div class="post-metadata">

### Author: ![gerhard](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/gerhard/32/119479_2.png) [@gerhard](https://meta.discourse.org/u/gerhard)
#### Post date: [4 september 2015 om 23:23 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/208 "2015-09-04T23:23:30Z")

</div>

The new menus still have some problems on Android 5 and Chrome (tested on my Nexus 5). They are a little bit too high.

 ![](https://global.discourse-cdn.com/meta/original/3X/c/7/c737c77153b5947e76ba1c6523fd1622e9bb05ab.png)  
 ![](https://global.discourse-cdn.com/meta/original/3X/d/7/d7872bcb5468b4dfdfad0303b60294bb5f71dd65.png)

And it’s impossible to scroll to the bottom of those menus. They always jump up a little bit and start scrolling the topic list in the background. Here’s a screenshot in landscape mode where I tried scrolling to the bottom of the menu…

 ![](https://global.discourse-cdn.com/meta/original/3X/3/8/3827aa8049d36ba08cbd6897c54ecfbd498d0661.png) 

Would it be possible to stop propagating scroll events to parent elements if the element that receives the scroll event has scrollbars? I have a similar problem on desktop when I scroll in the preview while creating a post. It always starts scrolling the visible posts whenever I reach the top or bottom of the preview.  
_Offtopic_: Bonus points if scrolling could be disabled while modal windows are shown.

---

<div class="post-metadata">

### Author: ![sam](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/sam/32/102149_2.png) [@sam](https://meta.discourse.org/u/sam)
#### Post date: [4 september 2015 om 23:28 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/209 "2015-09-04T23:28:00Z")

</div>

Same on iPhone chrome, iPhone safari is fine except for the real weird hover effect

---

<div class="post-metadata">

### Author: ![gerhard](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/gerhard/32/119479_2.png) [@gerhard](https://meta.discourse.org/u/gerhard)
#### Post date: [4 september 2015 om 23:33 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/210 "2015-09-04T23:33:54Z")

</div>

One more thing: Clicking on the Discourse logo in the header doesn’t close the Hamburger and profile menus. I’m quite sure that this used to work.

---

<div class="post-metadata">

### Author: ![tobiaseigen](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/tobiaseigen/32/539204_2.png) [@tobiaseigen](https://meta.discourse.org/u/tobiaseigen)
#### Post date: [5 september 2015 om 00:04 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/211 "2015-09-05T00:04:53Z")

</div>

crazy cool! 👍

---

<div class="post-metadata">

### Author: ![simon](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/simon/32/339122_2.png) [@simon](https://meta.discourse.org/u/simon)
#### Post date: [5 september 2015 om 00:21 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/212 "2015-09-05T00:21:54Z")

</div>

The problem with not being able to scroll to the bottom of the content seems to come from `$panelBody.height('100%')` in `menu-panel.js.es6` - that is setting the height to 100% of the content. The result of that is that some of the content may occur outside of the window. Subtracting the header height from the window height and setting panelBody to that seems to solve the problem `let panelHeight = (window).height() - 55;`

* * *

edit - this is wrong. There are a few things going on here, but this isn’t it.

---

<div class="post-metadata">

### Author: ![holden](https://avatars.discourse-cdn.com/v4/letter/h/e36b37/32.png) [@holden](https://meta.discourse.org/u/holden)
#### Post date: [5 september 2015 om 01:04 UTC](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519/213 "2015-09-05T01:04:24Z")

</div>

Oooh I dig it @Tom_Newsom !!

[Previous page](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519.md?page=9)

[Next page](https://meta.discourse.org/t/feedback-on-new-hamburger-and-user-menus/32519.md?page=11)
