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

[Refactor][GUI] Set static texts in .ui files + add missing tr() #1604

Merged

Conversation

random-zebra
Copy link

Set initial / static texts for widgets and dialogs in the .ui files instead of in the .cpp files, with notr where needed. Also add tr() where missing (ref: #1600).
Finally clean up styling (missing spaces, branches on newlines).

@random-zebra random-zebra added this to the 5.0.0 milestone May 6, 2020
@random-zebra random-zebra self-assigned this May 6, 2020
@random-zebra random-zebra force-pushed the 202005_GUI_translations-and-strings branch 4 times, most recently from 4856626 to a8f5b1f Compare May 6, 2020 22:12
furszy
furszy previously approved these changes May 8, 2020
Copy link

@furszy furszy left a comment

Choose a reason for hiding this comment

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

What a large PR, cool work.

Left few minor comments, none of them is a blocking one and can be tackled in another PR.

utACK a8f5b1f

src/qt/pivx/coldstakingwidget.cpp Outdated Show resolved Hide resolved
src/qt/pivx/forms/addresseswidget.ui Outdated Show resolved Hide resolved
src/qt/pivx/forms/dashboardwidget.ui Outdated Show resolved Hide resolved
src/qt/pivx/mnrow.cpp Outdated Show resolved Hide resolved
@random-zebra
Copy link
Author

All comments addressed

Copy link

@furszy furszy left a comment

Choose a reason for hiding this comment

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

re ACK cc60e50

Copy link
Collaborator

@Fuzzbawls Fuzzbawls left a comment

Choose a reason for hiding this comment

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

Nice one!

ACK cc60e50

@Fuzzbawls Fuzzbawls merged commit 8ec6bbe into PIVX-Project:master May 11, 2020
@random-zebra random-zebra deleted the 202005_GUI_translations-and-strings branch September 24, 2020 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants