New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Return empty space in tray menu #944

Closed
mysteriously opened this Issue Mar 4, 2016 · 8 comments

Comments

Projects
None yet
3 participants
@mysteriously

mysteriously commented Mar 4, 2016

Original message:

You removed empty space shown in tray menu, but you left it in other places, hmm... why?

http://i.imgur.com/gzC4c3m.jpg

@mysteriously mysteriously changed the title from Remove empty space shown on RMB click to Remove empty space shown in menu on RMB click Mar 4, 2016

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov

ameshkov Mar 5, 2016

Member

Or maybe we should return it to the tray menu?

Member

ameshkov commented Mar 5, 2016

Or maybe we should return it to the tray menu?

@mysteriously

This comment has been minimized.

Show comment
Hide comment
@mysteriously

mysteriously Mar 6, 2016

Hmm, that's interesting. Generally I have never really paid much attention to the left part of context menus. Now I see, it is a place for icons and seems to be set up by default, even if icons are not used. All apps currently running on my PC have this space to the left in tray menu.

mysteriously commented Mar 6, 2016

Hmm, that's interesting. Generally I have never really paid much attention to the left part of context menus. Now I see, it is a place for icons and seems to be set up by default, even if icons are not used. All apps currently running on my PC have this space to the left in tray menu.

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov

ameshkov Mar 6, 2016

Member

It's a pity Windows does not have some clear UI guidelines so we are always trying to figure out the right solution by ourselves.

If everybody leaves that empty place, I guess we should do the same.

Member

ameshkov commented Mar 6, 2016

It's a pity Windows does not have some clear UI guidelines so we are always trying to figure out the right solution by ourselves.

If everybody leaves that empty place, I guess we should do the same.

@ameshkov ameshkov added this to the 6.0 R2 milestone Mar 6, 2016

@ameshkov ameshkov added the UI label Mar 6, 2016

@ameshkov ameshkov changed the title from Remove empty space shown in menu on RMB click to Return empty space in tray menu Mar 6, 2016

@suhan3z

This comment has been minimized.

Show comment
Hide comment
@suhan3z

suhan3z Mar 23, 2016

Contributor

Fixed

Contributor

suhan3z commented Mar 23, 2016

Fixed

@suhan3z suhan3z closed this Mar 23, 2016

@mysteriously

This comment has been minimized.

Show comment
Hide comment
@mysteriously

mysteriously Mar 30, 2016

The empty space width in tray menu is not as wide as in other places.
http://i.imgur.com/cKgoiVX.png

mysteriously commented Mar 30, 2016

The empty space width in tray menu is not as wide as in other places.
http://i.imgur.com/cKgoiVX.png

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov

ameshkov Mar 31, 2016

Member

That's because AG uses .NET tray menu which is a bit different from native tray menu used by other apps. I have no idea why they did this, but that's how it is:)

Member

ameshkov commented Mar 31, 2016

That's because AG uses .NET tray menu which is a bit different from native tray menu used by other apps. I have no idea why they did this, but that's how it is:)

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov
Member

ameshkov commented Mar 31, 2016

@suhan3z please take a look at this:
http://stackoverflow.com/questions/5055433/how-to-display-a-windows-7-style-context-menu

Can we use old context menu instead?

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov

ameshkov Mar 31, 2016

Member

Moved it to a separate issue:
#1006

Member

ameshkov commented Mar 31, 2016

Moved it to a separate issue:
#1006

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