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

App strings reworked #277

Merged
merged 6 commits into from
May 11, 2023
Merged

App strings reworked #277

merged 6 commits into from
May 11, 2023

Conversation

comradekingu
Copy link
Contributor

@@ -8,19 +8,19 @@
},
"error": {
"label": {
"empty": "Category is Empty",
"empty": "Empty Category",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
"empty": "Empty Category",
"empty": "Empty category",

?

Copy link
Member

Choose a reason for hiding this comment

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

don't change this, this is a message shown to the user when a category is empty.

@@ -8,19 +8,19 @@
},
"error": {
"label": {
"empty": "Category is Empty",
"empty": "Empty Category",
Copy link
Member

Choose a reason for hiding this comment

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

don't change this, this is a message shown to the user when a category is empty.

Comment on lines 328 to 329
"continuous_horizontal_ltr": "Horizontal (LTR)",
"continuous_horizontal_rtl": "Horizontal (RTL)",
Copy link
Member

Choose a reason for hiding this comment

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

nice touch, maybe we could use emoji?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Certainly. It may also be possible to find something for "double page".

Copy link
Collaborator

Choose a reason for hiding this comment

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

do we really want to have the icon/emoji in the translations? wouldn't it be better to add it in the code?

Copy link
Contributor Author

@comradekingu comradekingu Apr 1, 2023

Choose a reason for hiding this comment

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

Would end up on the wrong side RTL if done on the backend?

Copy link
Collaborator

Choose a reason for hiding this comment

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

you're right, didn't think about that

},
"title": "Backup"
},
"label": {
"dark_theme": "Dark Theme",
"image_cache": "Use image cache",
"image_cache_description": "Disabling image cache makes images load faster if you have a slow disk, but uses it much more internet traffic in turn",
"image_cache_description": "Preloading images shows them quicker, but downloads more data.",
Copy link
Member

Choose a reason for hiding this comment

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

It was confusing as it was, now its worse

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm also unaware that the cache has something to do with preloading?

regarding the old description, I don't think any disk (unless broken) will actually have any impact.
so the only change I see, is that the images will always be freshly loaded from the source

Copy link
Collaborator

@schroda schroda left a comment

Choose a reason for hiding this comment

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

regarding the nouns, the correct spelling should be in lower case @AriaMoradi

"by_date_added": "By Date Added",
"by_last_read": "By Last Read",
"by_unread_chapters": "By Unread chapters"
"by_unread_chapters": "By Unread Chapters"
Copy link
Collaborator

Choose a reason for hiding this comment

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

imo that should be all lowercase letters (e.g. "By date added")

},
"title": "Backup"
},
"label": {
"dark_theme": "Dark Theme",
"image_cache": "Use image cache",
"image_cache_description": "Disabling image cache makes images load faster if you have a slow disk, but uses it much more internet traffic in turn",
"image_cache_description": "Preloading images shows them quicker, but downloads more data.",
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm also unaware that the cache has something to do with preloading?

regarding the old description, I don't think any disk (unless broken) will actually have any impact.
so the only change I see, is that the images will always be freshly loaded from the source

@schroda
Copy link
Collaborator

schroda commented May 10, 2023

  • resolved merge conflict
  • use emojis for reader types
  • updated image cache strings

@schroda schroda requested a review from AriaMoradi May 10, 2023 18:00
@schroda schroda merged commit b4b3dc5 into Suwayomi:master May 11, 2023
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.

4 participants