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

points as decimal separator instead of commas, and no thousands separator in layout composer #33727

Closed
stadtentwicklung opened this issue Jan 10, 2020 · 19 comments
Labels
Feature Request Print Layouts Related to QGIS Print Layouts, Atlas or Reporting frameworks

Comments

@stadtentwicklung
Copy link

Describe the bug

How to Reproduce

QGIS and OS versions

Additional context

@stadtentwicklung stadtentwicklung added the Bug Either a bug report, or a bug fix. Let's hope for the latter! label Jan 10, 2020
@stadtentwicklung stadtentwicklung changed the title Numbers with points as Numbers with points instead of commas and no thousands separator in QGIS 3.4.7 layout Jan 10, 2020
@stadtentwicklung
Copy link
Author

Attribute table (1) uses commas as the decimal marker, which is correct. But the layout (2) uses different notation, with points for decimal separators and no thousands seperator. The settings seems right (3).

1_SHP
2_Layout
3_Settings

@stadtentwicklung
Copy link
Author

@roya0045
Copy link
Contributor

Update your qgis and please report if this is still the case.

@stadtentwicklung
Copy link
Author

okay I'll do …
and by the way the QGIS opening window fits perfect in the default vlc player window...

Unbenannt

@gioman gioman added the Feedback Waiting on the submitter for answers label Jan 12, 2020
@gioman gioman changed the title Numbers with points instead of commas and no thousands separator in QGIS 3.4.7 layout points as decimal separator instead of commas, and no thousands separator in QGIS 3.4.7 layout composer Jan 12, 2020
@stadtentwicklung
Copy link
Author

as @roya0045 said, I updated QGIS from 3.4.7 to 3.4.15. The problem is still existing. See the screenshot. Issue again: points as decimal separator instead of commas, and no thousands separator in QGIS 3.4.15 layout composer (but in the "map-canvas" everything is fine -> see attachement please).

MADeira

@elpaso
Copy link
Contributor

elpaso commented Jan 27, 2020

This was never implemented.

I could work on it, but I'm not sure if @nyalldawson has plans.

Additionally, something that we might want to add is to optionally add the language information to the layout, so that when opened on QGIS instance with a different language setting (think about the server context where we want prints with a certain language configuration).

@elpaso elpaso added Feature Request and removed Bug Either a bug report, or a bug fix. Let's hope for the latter! labels Jan 27, 2020
@stadtentwicklung
Copy link
Author

So if I understand you right, the language information of content in the layout-composer is a individual one, free and independent? All Information after your third comma I do not understand.

@elpaso
Copy link
Contributor

elpaso commented Jan 27, 2020

So if I understand you right, the language information of content in the layout-composer is a individual one, free and independent?

It was never implemented: there is no locale support in layout composer tables.

All Information after your third comma I do not understand.

Sorry for not being clear, what I meant is that when (and if) it will be implemented, we will need to keep in mind that we might want to allow a different behavior than the most part of QGIS: instead of always getting locale information form user profile settings, we might want to be able to override the locale settings for the print layouts individually, just in case your project contains different layouts for different locales.

@stadtentwicklung
Copy link
Author

Thank you for your answer. I still not understand. The setting is in the canvas perfect and works fine with decimal commas and thousand separators. I assume, that the canvas setting shows the locale setting. This locale setting is not consider by the composer. The composer setting is not clear in my opinion. It should get the "locale canvas setting" too.

@elpaso
Copy link
Contributor

elpaso commented Jan 28, 2020

Thank you for your answer. I still not understand. The setting is in the canvas perfect and works fine with decimal commas and thousand separators. I assume, that the canvas setting shows the locale setting. This locale setting is not consider by the composer. The composer setting is not clear in my opinion. It should get the "locale canvas setting" too.

I agree. But the piece of code that does that was never written: it was never implemented, so technically it's not a bug.

@stadtentwicklung
Copy link
Author

Okay chrystal clear. How can I go on?

@elpaso
Copy link
Contributor

elpaso commented Jan 28, 2020

The usual way, you have a few options:

  • fix it yourself if you can
  • wait and hope that someone fixes it for you
  • launch a crowdfunding campaign to get it financed
  • contact a QGIS core developer and fund the development

@stadtentwicklung
Copy link
Author

The checkbox "System-Locale überschreiben (override)" is not activated (top of the pic). (But) All numbers are beautiful in the attribute table (the end of the pic). The default of "System-Locale" has access to the right settings. What are these settings? Is it a lot of work to implement that default setting in the composer? Where can I start?

@stadtentwicklung
Copy link
Author

1

@stadtentwicklung
Copy link
Author

Sorry my answer was not clear. I would try to fix it by myself and I was asking for an entry point. I decide to start with the GitHub repositories.

@gioman
Copy link
Contributor

gioman commented Feb 5, 2020

If the issue is valid/confirmed it must stay open.

@gioman gioman changed the title points as decimal separator instead of commas, and no thousands separator in QGIS 3.4.7 layout composer points as decimal separator instead of commas, and no thousands separator in layout composer Feb 5, 2020
@elpaso
Copy link
Contributor

elpaso commented Feb 5, 2020

@gioman if this is a bug or a feature request is debatable.
I think it's a bug, but it was never implemented in the first place.

IIRC @nyalldawson has been made aware of that and he had plans, but we spoke about that a long time ago, so I may be mistaken.

The fix can be trivial or complex depending how we decide to handle it (see my comments above regarding the server situation).

@gioman
Copy link
Contributor

gioman commented Feb 5, 2020

@gioman if this is a bug or a feature request is debatable.
I think it's a bug, but it was never implemented in the first place.

sure, but one or the other does not matter, if the bug/request is a valid one.

@github-actions
Copy link

The QGIS project highly values your report and would love to see it addressed. However, this issue has been left in feedback mode for the last 14 days and is being automatically marked as "stale".
If you would like to continue with this issue, please provide any missing information or answer any open questions. If you could resolve the issue yourself meanwhile, please leave a note for future readers with the same problem and close the issue.
In case you should have any uncertainty, please leave a comment and we will be happy to help you proceed with this issue.
If there is no further activity on this issue, it will be closed in a week.

@github-actions github-actions bot added the stale Uh oh! Seems this work is abandoned, and the PR is about to close. label Dec 25, 2020
@gioman gioman removed the Feedback Waiting on the submitter for answers label Dec 27, 2020
@gioman gioman added Print Layouts Related to QGIS Print Layouts, Atlas or Reporting frameworks and removed stale Uh oh! Seems this work is abandoned, and the PR is about to close. labels Dec 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request Print Layouts Related to QGIS Print Layouts, Atlas or Reporting frameworks
Projects
None yet
Development

No branches or pull requests

4 participants