Skip to content
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

Request: Arranger Tree #305

Closed
newboerg opened this issue Dec 12, 2016 · 7 comments
Closed

Request: Arranger Tree #305

newboerg opened this issue Dec 12, 2016 · 7 comments
Assignees

Comments

@newboerg
Copy link

newboerg commented Dec 12, 2016

It would be nice if the Structorizer had a Tree-View of the current "Project".

If you have more then 2 bigger Diagrams in the Arranger you can barely review anything
problemclipboard02

I am proposing something like this:
proposal

@codemanyak
Copy link
Collaborator

codemanyak commented Dec 12, 2016

You are right, at least a diagram index view showing the names (and possibly paths ) of the diagrams parked in Arranger would really be helpful.
Interestingly, about ten years ago, the early Structorizer version 1.26 seems to have conveyed a project concept already. I noticed that quite recently on scrolling through the changelog. Apparently, the subsequent Structorizer reimplementations did not re-introduce it. Maybe because a mandatory project context complicates the firts steps for a beginner. But a missing project context limits the usability for the advanced...
The Arranger index list might be a first helpful low-impact approach to address the needs of both. I think it might work as follows:

  1. It appears only after some diagram has been sent to the Arranger.
  2. It may be hidden or shown on demand, similar to the report list of the Analyser, but docked at the left or (possibly better) right margin.
  3. The list will be sorted by diagram name or better signature (name + argument count).
  4. A single click on an index entry would scroll the Arranger to the respective diagram and put it at top level.
  5. A double click would fetch it back to the work area of the Structorizer, replacing the currently edited diagram (after having offered to save its recent changes).

A further enhancement might be to pop up the file path of the diagram while the curser is hovering over it in the Arranger.
What do you think? Would that help on the short term while we think about a possible specification of a project environment?

@newboerg
Copy link
Author

This sounds exactly like what i had in mind :-)
The offering of save is especially appealing.
Maybe add an Auto-Save-On-Change Option somewhere.

@codemanyak
Copy link
Collaborator

The offering of save is especially appealing.
Maybe add an Auto-Save-On-Change Option somewhere.

Oops, I thought the Structorizer does already offer to save the current changes whenever another diagram is pushed into it? (I mentioned this in the "specification" above only for completeness).
Can you describe a situation where unsaved changes are overridden without a callback?
Thanks for your proposals and comments.

@newboerg
Copy link
Author

newboerg commented Dec 12, 2016

Well, it happens when leaving a diagramm(through call) which has not been saved prior to execution.

func

It should "Auto-Save" in that case, at least make it opt-in. its very annoying, because in case of using turtle it looses focus.

@codemanyak
Copy link
Collaborator

codemanyak commented Dec 13, 2016

It should "Auto-Save" in that case, at least make it opt-in. its very annoying, because in case of using turtle it looses focus.

This hasn't been implemented in version 3.25-10 yet, I will file it as an extra enhancement issue (#310) since it requires some wide-spread modifications.

@codemanyak
Copy link
Collaborator

codemanyak commented Dec 15, 2016

The focus problems described in #312 have successfully been addressed and will be fixed with the next version (3.25-11) very soon.

codemanyak added a commit that referenced this issue Dec 16, 2016
fesch added a commit that referenced this issue Dec 16, 2016
Improves #305, fixes#310, fixes parts of #311, fixes #312 -> version 3.25-11
codemanyak added a commit that referenced this issue Dec 21, 2016
- Issue #305: Clicking into the Arranger index should force Arranger visibility
- Enh. #305: Key binding <del> added to Arranger index (removes diagram)
- Enh. #267: New Analyser check against ambiguous CALLs (multiple matches)
- Enh. #314: File I/O API prototype
- Bugfix #317: Color of empty sequences (like empty FALSE branch) now saved
@codemanyak
Copy link
Collaborator

A keybinding for the <del> key has been added, allowing now to remove a diagram selected in the Arranger index from the Arranger. This way, the Arranger can easily be cleared from Structorizer. (Due with version 3.25-12 or release 3.26.)

codemanyak added a commit that referenced this issue Dec 29, 2016
- Arranger index now marks "dirty" diagrams ('*' on unsaved changes)
- Executor aborts on ambiguous CALLs
codemanyak added a commit that referenced this issue Jan 13, 2017
…fication on diagram name or dirtiness change)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants