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

Update property list (language bar) #122

Merged
merged 1 commit into from
Aug 25, 2017
Merged

Conversation

hiunnhue
Copy link
Collaborator

This commit contains the following modifications:

  1. Fix: the property list (language bar) does not update when
    toggling Chi/Eng Mode or Fullwidth/Halfwidth Shape via the
    systray icon.

  2. Add a setup button to the ibus indicator (and the language
    bar), so that users can open the setup window more quickly.
    In order to have consistent style with other buttons on
    bar, use property symbol instead of icon.

  3. Make the naming of prop_name more consistent:
    "chiEng_prop", "alnumSize_prop", and "setup_prop".

  4. Update property labels and tooltips.

This commit contains the following modifications:

1. Fix: the property list (language bar) does not update when
toggling Chi/Eng Mode or Fullwidth/Halfwidth Shape via the
systray icon.

2. Add a setup button to the ibus indicator (and the language
bar), so that users can open the setup window more quickly.
In order to have consistent style with other buttons on
bar, use property symbol instead of icon.

3. Make the naming of prop_name more consistent:
"chiEng_prop", "alnumSize_prop", and "setup_prop".

4. Update property labels and tooltips.
@hiunnhue
Copy link
Collaborator Author

修改後的語言工具列:

language_bar

修改後的 ibus indicator :

ibus_indicator

@definite definite merged commit add50d6 into chewing:master Aug 25, 2017
@definite
Copy link
Member

Thanks for your contribution!

@hiunnhue hiunnhue deleted the setup_prop branch August 25, 2017 13:50
@hiunnhue
Copy link
Collaborator Author

Thanks for reviewing and merging ! :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants