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.

Sign up for GitHub

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

Jump to bottom

Enhancement: Additional CLI options to control rendered output -no-fields #53

Closed
jfeliu007 opened this issue Oct 24, 2019 · 2 comments · Fixed by #59
Closed

Enhancement: Additional CLI options to control rendered output -no-fields #53

jfeliu007 opened this issue Oct 24, 2019 · 2 comments · Fixed by #59
Assignees
@jfeliu007
Labels
enhancement New feature or request
Projects
Release v1.2.2

Comments

@jfeliu007
Copy link
Owner

Provide the CLI flag
-no-fields
To hide/exclude Classes' "fields"
This would be useful for simplifying/filtering the output. E.g. Reducing the size of the Class nodes (by including less intra-Class detail) simplifies the graph layout, and makes it easier to trace inter-Class relationships during an API review of the coupling between Classes.
@jfeliu007 jfeliu007 added the enhancement New feature or request label Oct 24, 2019
@jfeliu007 jfeliu007 added this to To do in Release v1.2.2 Oct 24, 2019
@jfeliu007 jfeliu007 moved this from To do to In progress in Release v1.2.2 Oct 24, 2019
@jfeliu007 jfeliu007 self-assigned this Oct 24, 2019
@jfeliu007
Copy link
Owner Author

I decided to name this flag -hide-fields it looks more like an action. The result would be to add the hide fields directive at the bottom of the diagram so that no fields render, but the fields remain in the diagram. The user can manually set this up easily if desired.

jfeliu007 pushed a commit that referenced this issue Oct 24, 2019
Enhancement: Additional CLI options to control rendered output -no-fi…
c984175 
…elds

Fixes #53
Added the -hide-fields flag to the CLI and added support in the RenderingOptions to pass this directive. The result will be the hide fields directive added to the bottom of the diagram
@jfeliu007 jfeliu007 mentioned this issue Oct 24, 2019
Merged
This was referenced Oct 24, 2019
Closed
Closed
@jfeliu007 jfeliu007 moved this from In progress to Review in Release v1.2.2 Oct 24, 2019
@jfeliu007 jfeliu007 moved this from Review to Done in Release v1.2.2 Oct 24, 2019
@jfeliu007 jfeliu007 mentioned this issue Oct 24, 2019
Closed
@Justin-W
Copy link

Works great! Thanks!

Tested #53 and #54 together with goplantuml -recursive -hide-fields -hide-methods .

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

@jfeliu007 jfeliu007

Labels
enhancement New feature or request
Projects
No open projects
Release v1.2.2
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Enhancement: Additional CLI options to control rendered output -no-fi… jfeliu007/goplantuml
2 participants
@Justin-W @jfeliu007