Move GUI logic out of the main function

Bug #1445838 reported by Antoine Proulx
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Wishlist
Unassigned

Bug Description

Hi,

I hope this is the right place to submit patches to Inkscape. In order to understand how does Inkscape work (because I want to contribute to it), I move out some GUI logic from the main to other functions. I think it is now clearer like this.

Revision history for this message
Antoine Proulx (magicienap) wrote :
su_v (suv-lp)
tags: added: cli code-design
Changed in inkscape:
importance: Undecided → Wishlist
Revision history for this message
Qantas94Heavy (qantas94heavy) wrote :

Hi there, thanks for the patch and apologies for not noticing this earlier.

It appears that this has already been refactored in current Inkscape master: https://gitlab.com/inkscape/inkscape/blob/master/src/inkscape-application.cpp

Obviously this wasn't around back then, but you can now submit code changes as a merge request on GitLab: https://gitlab.com/inkscape/inkscape

If you have any questions, feel free to ask!

Changed in inkscape:
status: New → Fix Committed
milestone: none → 1.0
Max Gaukler (mgmax)
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.