Mattermost-server: Improve the Main Menu popover position/placement

Created on 2 Apr 2020  路  12Comments  路  Source: mattermost/mattermost-server

Opening the Main Menu places the popover menu in what seems to be a random location. It's not under the menu icon or the LHS header. This creates confusion since the entire header is clickable in the LHS.

Improvement

  • Align the dropdown menu with the users avatar in the sidebar
  • Remove the main menu tooltip on hover, the hover state is enough of an affordance

image-2020-03-02-14-56-29-761


If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-22828

AreEnd User Feature Medium Help Wanted PR Exists TecReactJS

All 12 comments

Hi, could I work on this issue?

Sure, that would be great @Qovaros :+1:

While solving this issue, I had two thoughts.

  1. Set status dropdown should probably also be moved to align with user icon. Right now it is not aligned the same as main menu should be as the output of this story.
    Current:
    Screenshot from 2020-04-10 15-28-09
    Expected:
    Screenshot from 2020-04-10 15-22-01

  2. Maybe set status dropdown should display tooltip on hover. Right now it only changes icon in lower right corner.
    Screenshot from 2020-04-10 15-26-02

@andrewbrown00 Do you have thoughts on this?

Hi @Qovaros thanks for taking on this improvement 馃槃

  1. I agree, we should be consistent and align both the main menu and the status menu to open in the same position. The are some close with this improvement anyway.
  2. Because the status dropdown is generally interacted with quickly, I would avoid using the tooltip as an additional affordance. Plus, the control is small and the tooltip may be intrusive in this context.

One thing we could improve, _if it doesn't impact the scope of this ticket_, is to hide the native tooltip with the icon name 馃槈
image

Thanks for response @andrewbrown00. I feel like it could be included in the scope of this ticket.
During development I had some other thoughts. With other chevrons there also is a tooltip on hover.

For example in mobile view
Screenshot from 2020-04-15 02-02-51

or in member modal
Screenshot from 2020-04-15 01-09-39

Should it be like that?

Welcome @Qovaros and no, those tooltips shouldn't display. They can be removed like you did for the status chevron.

Thanks @andrewbrown00 for review. I think it would be better to create a separate PR for other chevrons. I can take care of that.

@andrewbrown00 while removing this tooltip from dropdown menu chevron I noticed, there are some other icons which also have this tooltip. Shouldn't those be removed too? If yes, maybe we should create an issue about taking care of all such things in webapp?
Screenshot from 2020-05-10 20-30-33
Screenshot from 2020-05-10 20-29-51

Thanks @Qovaros for bringing this to our attention 馃憤

Would you be interested in creating a separate issue and removing all tooltips from the icons? Not sure of it's easier for you to find them in the code or you'd like us to take an inventory of where they are in the product first?

cc @esethna this would be a nice improvement to clean up unnecessary tooltips

I can find and remove them myself. I would just need some confirmation which should and which shouldn't be removed. If you want me to I can create an issue, link it here and list all unnecessary tooltips I found.

Sure, thanks @Qovaros!

Was this page helpful?
0 / 5 - 0 ratings