Skip to content
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.

Sign up for GitHub

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

Jump to bottom

please revert to normal menu alignment #2756

Closed
mc-butler opened this issue Mar 21, 2012 · 6 comments
Closed

please revert to normal menu alignment #2756

mc-butler opened this issue Mar 21, 2012 · 6 comments
Assignees
@aborodin
Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress ver: 4.8.2 Reproducible in version 4.8.2
Milestone
4.8.3

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/2756
Reporter onlyjob (@onlyjob)

In MC 4.8.2 one of the changes was new "justified" menu alignment.

This is creating discomfort, because such changes in MC presentation are unexpected (and unwanted).

More importantly, this is a serious diversion from common GUI standards where vast majority of applications have top level horizontal menu left aligned.

I believe many people (including myself) will consider this to be regression.

Please restore traditional menu alignment.

Thanks.
@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Mar 21, 2012 at 12:53 UTC (comment 1)

  • Status changed from new to accepted
  • Version changed from master to 4.8.2
  • Owner set to andrew_b

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Mar 22, 2012 at 6:29 UTC (comment 2)

  • Branch state changed from no branch to on review

Branch: 2756_menu (parent: master).
[1cfdbd53c93d8f2847a037aacd3abc5953d6a433]

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Mar 22, 2012 at 9:39 UTC (comment 3)

  • Votes set to slavazanko

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Mar 24, 2012 at 18:56 UTC (comment 4)

  • Votes changed from slavazanko to slavazanko angel_il
  • Branch state changed from on review to approved

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Mar 25, 2012 at 6:38 UTC (comment 5)

  • Branch state changed from approved to merged
  • Votes changed from slavazanko angel_il to committed-master
  • Resolution set to fixed
  • Status changed from accepted to testing

Merged to master: [4cc0aef].

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Mar 25, 2012 at 6:39 UTC (comment 6)

  • Status changed from testing to closed

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

@aborodin aborodin

Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress ver: 4.8.2 Reproducible in version 4.8.2
Milestone
4.8.3
Development

No branches or pull requests
2 participants
@aborodin @mc-butler