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

feat: change title from Nim Status Client to Status Desktop #829

Merged
merged 2 commits into from
Sep 7, 2020

Conversation

dzuqe
Copy link
Contributor

@dzuqe dzuqe commented Sep 4, 2020

fixes #392

Change the title "Nim Status Client" to "Status Desktop in the translations, main.qml and nim-status.desktop files

@dzuqe dzuqe changed the title change title from Nim Status Client to Status Desktop feat: change title from Nim Status Client to Status Desktop Sep 4, 2020
@status-im-auto
Copy link
Member

status-im-auto commented Sep 4, 2020

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 38cebfc #1 2020-09-04 16:17:20 ~6 min macos 📦dmg
✔️ 38cebfc #1 2020-09-04 16:21:44 ~11 min linux 📦App
✔️ 6a0ff70 #2 2020-09-04 21:08:07 ~4 min macos 📦dmg
✔️ 6a0ff70 #2 2020-09-04 21:11:17 ~7 min linux 📦App

//% "Nim Status Client"
Qt.application.name = qsTrId("nim-status-client")
//% "Status Desktop"
Qt.application.name = qsTrId("status-desktop")
Copy link
Member

@richard-ramos richard-ramos Sep 4, 2020

Choose a reason for hiding this comment

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

This needs to match the message-id in the .ts files: <message id="nim-status-client">

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey. When i change this string to the id nim-status-client, the running application reverts back to "Nim Status Desktop" even though the source for the id nim-status-client (in all the translation files) is in fact Status Desktop. I've decided to add a new commit that rather changes all the message ids to status-desktop for consistency sake.

@iurimatias iurimatias merged commit 464a201 into status-im:master Sep 7, 2020
micieslak pushed a commit that referenced this pull request Sep 21, 2022
…829)

- change StatusSlider to be vertically centered

Needed for #6725
micieslak pushed a commit that referenced this pull request Sep 21, 2022
…829)

- change StatusSlider to be vertically centered

Needed for #6725
micieslak pushed a commit that referenced this pull request Sep 21, 2022
…829)

- change StatusSlider to be vertically centered

Needed for #6725
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.

rename app name/title to say "Status Desktop"
4 participants