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

[FEATURE] Hide data types #333

Closed
ian-barker opened this issue Jan 7, 2025 · 7 comments · Fixed by #366
Closed

[FEATURE] Hide data types #333

ian-barker opened this issue Jan 7, 2025 · 7 comments · Fixed by #366

Comments

@ian-barker
Copy link

To cater for conceptual and certain logical models, the ability to hide data types from the ERD visualisation.
Perhaps through a simple toggle in the view menu.
This would leave only the field names visible in the tables.

@flashpunch0-0
Copy link
Contributor

Yeah it kinda looks different
without datatypes mentioned it would look clean

@flashpunch0-0
Copy link
Contributor

what do you think removing it would be a great idea?
as on hover it does show the datatype of that field

@ian-barker
Copy link
Author

@flashpunch0-0 I think if there was a toggle to hide the datatypes then it would solve the problem for me.
I'm not much of a UX coder though so I wouldn't really know how to add this in.
I was hoping someone more knowledgeable could have a go at it. :)

@flashpunch0-0
Copy link
Contributor

@ian-barker got it
I did find the piece of code, just taking a look at possible places to add the toggle button.

according to you what would be an ideal place like in the table itself or in the sidebar?

@ian-barker
Copy link
Author

@flashpunch0-0 I was thinking a sensible place would be in the "view" menu along the top.

@flashpunch0-0
Copy link
Contributor

hmm makes sense but I think if we give option right in the table it would be a better thing
let's see, I am figuring out a way

@flashpunch0-0
Copy link
Contributor

@1ilit I have created another pull request solving the issue
please have a look let me know if any changes are required

#366

@1ilit 1ilit linked a pull request Mar 16, 2025 that will close this issue
@1ilit 1ilit closed this as completed Mar 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants