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

enhance: add tooltip hint for prefered-file-paste #9564

Merged
merged 2 commits into from Jun 2, 2023

Conversation

Bad3r
Copy link
Collaborator

@Bad3r Bad3r commented Jun 1, 2023

image

I don't know if the message is too long. I used an image as the most common example.

@Bad3r Bad3r force-pushed the enhance/settings-tooltips branch 2 times, most recently from f17b14d to 9fb39be Compare June 1, 2023 22:14
Copy link
Collaborator

@logseq-cldwalker logseq-cldwalker left a comment

Choose a reason for hiding this comment

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

@Bad3r Looking good. Just one request and this should be good to go

@@ -299,6 +299,7 @@
:settings-page/custom-configuration "Individuelle Konfiguration"
:settings-page/custom-date-format "Bevorzugtes Datumsformat"
:settings-page/custom-date-format-warning "Neuindizierung erforderlich! Vorhandene Journal-Verweise würden nicht mehr funktionieren!"
:settings-page/preferred-pasting-file-hint "When enabled, pasting an image from the internet will download and insert the image. When disabled, it will paste the link to the image."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Worth removing all the non-english English entries like this. A language file should only contain entries for its language. Our translation system already handles defaulting to English when a key doesn't exist in another language

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@logseq-cldwalker sounds good will remove it. The reason I thought it should be included is to make it easier for translators to know what still needs to be translated.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Gotcha. Most translators rely on this workflow which depends on entries being missing in a language

We should get rid of English entries in any non-English file as they are redundant and confusing translators. I'll try to knock this out next week but PR is welcome for this as well

@Bad3r Bad3r force-pushed the enhance/settings-tooltips branch 2 times, most recently from bdcd83b to 3838659 Compare June 2, 2023 21:47
@Bad3r Bad3r force-pushed the enhance/settings-tooltips branch from 3838659 to 65cf1ee Compare June 2, 2023 21:49
This reverts commit 65cf1ee.
Copy link
Collaborator

@logseq-cldwalker logseq-cldwalker left a comment

Choose a reason for hiding this comment

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

@Bad3r Thanks for noticing this and making it more user friendly! 👍 🚢

@logseq-cldwalker logseq-cldwalker merged commit 9cb3fa2 into logseq:master Jun 2, 2023
5 of 6 checks passed
@Bad3r Bad3r deleted the enhance/settings-tooltips branch June 3, 2023 01:47
Bad3r added a commit to Bad3r/Logseq that referenced this pull request Jun 4, 2023
* Remove deprecated :editor/command-trigger option

This option has been deprecated for a couple months

* enhance: auto pan

* enhance: naming

* fix: e2e tests

* enhance: support whiteboard translations

* fix: add whitespace

* Update valid dicts lint and delete an unused key

* enhance: add tooltip hint for prefered-file-paste (logseq#9564)

* enhance: add tooltip hint for prefered-file-paste

* Revert dict changes

This reverts commit 65cf1ee.

* chore: Update bug_report.yaml to reassure no question (logseq#9572)

---------

Co-authored-by: Gabriel Horner <gabriel@logseq.com>
Co-authored-by: Konstantinos Kaloutas <konstantinos@logseq.com>
Co-authored-by: Junyi Du <junyidu.cn@gmail.com>
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.

None yet

2 participants