That's the same problem as with the clipboard, but thistime, Krzysztofs return-solution won't work ;) . The patch is essentially the same I appended in this bugreport, I attached the version adjusted for grids.
Regards, Markus
-----Ursprüngliche Nachricht----- Von: suv@...2204... [mailto:suv@...2204...] Im Auftragvon su_v Gesendet: Freitag, 20. September 2013 13:56 An: Krzysztof Kosiński Cc: Markus Engel; inkscape-devel Betreff: Re: [Inkscape-devel] Cppify Branch Review
On 2013-09-19 04:09 +0200, Krzysztof Kosiński wrote:
There are basically two issues which stand out:
- Inconsistent whitespace
- Use of explicit this-> instead of markingmember variables with a
leading underscore.
However, I think these two stylistic problems can befixed later and it's better to commit now than to wait and let thisimportant work get out of sync with trunk.
I committed the merge as revision 12532.
After the merge in r12532 'File > Document Properties> Grids' no longer display the settings for defined grids (works withr12531, fails with r12532).
Steps to reproduce: 1) launch Inkscape (default prefs, default new doc) 2) toggle on grid (menu 'View > Grid') to create adefault grid 3) open 'File > Document Properties > Grids' toedit the settings
Expected result: The settings for the new grid are accessible on the'Grids' page
Actual result: The 'Defined grids' section is empty.
Warning on the console: ** (inkscape-12532:13355): WARNING **: TypeNotRegisteredexception: inkscape:grid
(tested and confirmed on Ubuntu 12.10 and OS X 10.7.5)