After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 723279 - Add champlain_view_get_bounding_box_for_zoom_level
Add champlain_view_get_bounding_box_for_zoom_level
Status: RESOLVED FIXED
Product: libchamplain
Classification: Core
Component: view
unspecified
Other Linux
: Normal normal
: ---
Assigned To: libchamplain-maint
libchamplain-maint
Depends on:
Blocks: 722865
 
 
Reported: 2014-01-30 12:13 UTC by Jonas Danielsson
Modified: 2014-02-05 19:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
view: Add get_bounding_box_for_zoom_level (6.24 KB, patch)
2014-01-30 12:14 UTC, Jonas Danielsson
committed Details | Review

Description Jonas Danielsson 2014-01-30 12:13:28 UTC
A function like this can be useful when trying to determine what zoom-level to use If, for instance, you want to make sure something is covered at a certain zoom-level.
Comment 1 Jonas Danielsson 2014-01-30 12:14:05 UTC
Created attachment 267626 [details] [review]
view: Add get_bounding_box_for_zoom_level

This function is useful for determining what zoom-level to
use for the view.
Comment 2 Zeeshan Ali 2014-02-03 17:52:51 UTC
Review of attachment 267626 [details] [review]:

I was hoping that Jiri will have time to review this one but seems not. Looks good to me.
Comment 3 Jonas Danielsson 2014-02-04 11:02:08 UTC
Attachment 267626 [details] pushed as 13a4a9b - view: Add get_bounding_box_for_zoom_level
Comment 4 Jiri Techet 2014-02-05 18:05:25 UTC
Was at FOSDEM and didn't have much time there. Looks good to me, just Since should be 0.12.6 in the docstring.
Comment 5 Zeeshan Ali 2014-02-05 19:12:36 UTC
(In reply to comment #4)
> Was at FOSDEM and didn't have much time there.

So were we! :)