Skip to content
This repository has been archived by the owner on Dec 14, 2021. It is now read-only.

Remember graph zoom level on close and restart #801

Closed
egraether opened this issue Nov 26, 2019 · 4 comments · Fixed by #1099
Closed

Remember graph zoom level on close and restart #801

egraether opened this issue Nov 26, 2019 · 4 comments · Fixed by #1099
Assignees
@diohabara
Labels
feature good first issue ideal for first-time contributors

Comments

@egraether
Copy link
Contributor

The current graph zoom level should be stored to the ApplicationSettings, but not displayed in the preferences dialog.

Implementation suggestion:

  • in ApplicationSettings add getter and setter for zoom level
  • in QtGraphicsView store and load this new setting
@egraether egraether added feature good first issue ideal for first-time contributors labels Nov 26, 2019
@diohabara
Copy link

diohabara commented Apr 27, 2020 •
edited
Loading

@egraether Hi, I'd like to fix this issue.
Can you assign me?

@egraether
Copy link
Contributor Author

Sure. Let me know if you have questions!

@miltolstoy miltolstoy mentioned this issue Oct 22, 2020
Merged
mlangkabel pushed a commit that referenced this issue Oct 23, 2020
@miltolstoy
logic: remember graph zoom level on restart (issue #801) (#1099)
ad9d8d7
@mlangkabel
Copy link
Contributor

Implemented via merge of pull request #1099. Thanks @miltolstoy!

@miltolstoy
Copy link
Contributor

My pleasure.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@diohabara diohabara

Labels
feature good first issue ideal for first-time contributors
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: remember graph zoom level on restart miltolstoy/Sourcetrail
4 participants
@egraether @mlangkabel @diohabara @miltolstoy