Skip to content

Conversation

@koppor
Copy link
Member

@koppor koppor commented May 10, 2025

Minor localization fixes

Mandatory checks

  • I own the copyright of the code submitted and I license it under the MIT license
  • [/] Change in CHANGELOG.md described in a way that is understandable for the average user (if change is visible to the user)
  • [/] Tests created for changes (if applicable)
  • [/] Manually tested changed features in running JabRef (always required)
  • [/] Screenshots added in PR description (if change is visible to the user)
  • [/] Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
  • [/] Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

new Label(Localization.lang("A new version of JabRef is available!")),
new Label(Localization.lang("Latest version") + ": " + latestVersion.getFullVersion()),
new Label(Localization.lang("Installed version") + ": " + currentVersion.getFullVersion()),
new Label(Localization.lang("Latest version: %0", latestVersion.getFullVersion())),
Copy link

Choose a reason for hiding this comment

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

The label text ends with a colon, which should be avoided as per guidelines. It should not end with a colon.

new Label(Localization.lang("Latest version") + ": " + latestVersion.getFullVersion()),
new Label(Localization.lang("Installed version") + ": " + currentVersion.getFullVersion()),
new Label(Localization.lang("Latest version: %0", latestVersion.getFullVersion())),
new Label(Localization.lang("Installed version: %0", currentVersion.getFullVersion())),
Copy link

Choose a reason for hiding this comment

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

The label text ends with a colon, which should be avoided as per guidelines. It should not end with a colon.

@Siedlerchr Siedlerchr enabled auto-merge May 10, 2025 15:40
@Siedlerchr Siedlerchr added this pull request to the merge queue May 10, 2025
Merged via the queue into main with commit ec396a8 May 10, 2025
2 checks passed
@Siedlerchr Siedlerchr deleted the fix-localization branch May 10, 2025 16:32
@github-actions
Copy link
Contributor

The build for this PR is no longer available. Please visit https://builds.jabref.org/main/ for the latest build.

Siedlerchr added a commit that referenced this pull request May 12, 2025
…ddLobidIsbnFetcher

* 'addLobidIsbnFetcher' of github.com:JabRef/jabref: (26 commits)
  Bump com.konghq:unirest-java-core from 4.4.6 to 4.4.7 (#13104)
  Bump ai.djl:bom from 0.32.0 to 0.33.0 (#13106)
  Bump org.openrewrite.rewrite from 7.5.0 to 7.6.1 (#13105)
  Bump com.tngtech.archunit:archunit-junit5-engine from 1.4.0 to 1.4.1 (#13103)
  Bump com.konghq:unirest-modules-gson from 4.4.6 to 4.4.7 (#13102)
  Bump jablib/src/main/resources/csl-styles from `b8070a7` to `9b777f6` (#13101)
  Refine http sever doc (#13100)
  Add type "Bug" (#13099)
  Fix dash
  feat: select correct tab for creation of new entry (#13087) (#13091)
  Add missing empty line
  Fix notarization flag (#13095)
  Add more options to ADR-044 (#13098)
  Update year (#13097)
  Add ADR for migration from Apache Commons to Picocli (#13096)
  fix-localization (#13092)
  Fix condition for "comment-on-pr"
  fix-git-version (#13093)
  hotfix: enable ea build also on label added
  Check label "dev: binary" for upload to builds.jabref.org (#13086)
  ...
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.

3 participants