https://bugs.kde.org/show_bug.cgi?id=473636

            Bug ID: 473636
           Summary: Percent values are not localisable
    Classification: Applications
           Product: gwenview
           Version: unspecified
          Platform: Other
                OS: Other
            Status: REPORTED
          Severity: normal
          Priority: NOR
         Component: general
          Assignee: gwenview-bugs-n...@kde.org
          Reporter: emir_s...@icloud.com
  Target Milestone: ---

Being an image viewer, Gwenview employs a quite a bit of number of percent
values throughout the application. However, the i18n of these values are
problematic, and does not allow modification for all use cases.

For instance, Turkish and some other languages use a percent format of %100, in
which the sign precedes the number. In Gwenview, the implementations mostly use
QLocale().percent(), which looks like nothing more than a percent sign itself,
and does not allow any locale-specific modification. Since Gwenview uses the
suffix-route to add the percent sign, there is no way to prepend the sign to
the number.

Places I've encountered the percent uses:
1. Bottom bar zoom slider. I wanted to fix this, but the implementation is not
trivial for me to understand it just yet, and I'm not comfortable touching it
either. Recently, Okular and Cantor had a similar issue fixed about the
percentage representation, it might be useful for reference.
2. Application title bar. I couldn't find the place that this is set, but it's
probably the same value as the combo box.
3. Various spin boxes. Unfortunately, Qt does not allow localising this, but
Lukas Sommer had this neat implementation[1] for Perceptual Color. It should be
fairly easy to adopt to Gwenview.

It would be great to have this fixed, since Gwenview is distributed with most
distributions, and fairly widely used. Looks really out-of place when compared
with the rest of the system.

[1] https://invent.kde.org/libraries/perceptualcolor/-/issues/1#note_654836

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to