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.

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

Keymaps: adjust keybindings-widget column spacing on firefox #6989

Merged
merged 1 commit into from
Jan 29, 2020

Conversation

Anasshahidd21
Copy link
Contributor

What it does

Fixes: 6117

When opened in firefox, there is a large column for action buttons shown, reduced the size of the column just so it can be made consistent with other browsers.

Signed-off-by: Muhammad Anas Shahid muhammad.shahid@ericsson.com

How to test

  • Try to open keyboard shortcuts on Firefox and Google Chrome(File>Settings>Open Keyboard Shortcuts )

  • Identify the difference in the size of the first column

  • This issue fixes the width of the column in Firefox and makes it equal with that of Chrome

Review checklist

Reminder for reviewers

@akosyakov akosyakov added the keybindings issues related to keybindings label Jan 29, 2020
When opened in firefox, there is a large column for action buttons shown, reduced the size of the column just so it can be made consistent with other browsers.

Issue ID: 6117

Signed-off-by: Muhammad Anas Shahid <muhammad.shahid@ericsson.com>
@Anasshahidd21
Copy link
Contributor Author

@vince-fugnitto I made the change, and im ready for a re-review

Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change looks good to me 👍

I verified the keybindings-widget using:

  • Firefox
  • Chrome
  • Safari

@vince-fugnitto vince-fugnitto merged commit 049fb52 into eclipse-theia:master Jan 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keybindings issues related to keybindings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[keymaps] adjust keybindings-widget column spacing on firefox
3 participants