-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
QGIS polygon Centroid (NewLabeling bug etc) #14990
Comments
Author Name: Serge Dikiy (Serge Dikiy)
|
Author Name: Serge Dikiy (Serge Dikiy) I think the patch should work, but I have no way to verify this. |
Author Name: Jürgen Fischer (@jef-n)
|
Author Name: Giovanni Manghi (@gioman)
|
Author Name: Giovanni Manghi (@gioman) Can you please detail/describe better the issue and the proposed solution?
|
Author Name: Serge Dikiy (Serge Dikiy) Giovanni Manghi wrote:
to understand the solution, please read this: |
Author Name: Giovanni Manghi (@gioman) Serge Dikiy wrote:
for me is ok :) I just asked to let people understand better what part of qgis is affected. |
Author Name: Serge Dikiy (Serge Dikiy) Giovanni Manghi wrote:
wise will understand :-) |
Author Name: Giovanni Manghi (@gioman)
it wasn't my intention to be "boring", the title and description are just not very explicit. By the way, thanks for the "RTFM" :) |
Author Name: Serge Dikiy (Serge Dikiy) http://en.wikipedia.org/wiki/User:Sdikiy/sandbox/PoligonCentroid |
Author Name: Paolo Cavallini (@pcav)
|
Author Name: Serge Dikiy (Serge Dikiy) fixed by @dakcarto #201 (comment) |
Author Name: Paolo Cavallini (@pcav)
|
Author Name: Serge Dikiy (Serge Dikiy)
Original Redmine Issue: 5256
Affected QGIS version: master
In QGIS Core I find code looks like this:
http://pastebin.com/qRcAa9GC
I think it code must be
http://pastebin.com/kFzgB0Mn
Same issue I open for OpenLayers
openlayers/openlayers#361
Related issue(s): #14535 (relates)
Redmine related issue(s): 4634
The text was updated successfully, but these errors were encountered: