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

re-work the "property panel" #52

Open
nicolasaunai opened this issue Nov 3, 2022 · 2 comments · May be fixed by #94
Open

re-work the "property panel" #52

nicolasaunai opened this issue Nov 3, 2022 · 2 comments · May be fixed by #94
Milestone

Comments

@nicolasaunai
Copy link

nicolasaunai commented Nov 3, 2022

  • put section titles (custom and global) in bold or something that let them easily distinguished from field names
  • maybe remove the UUID, the user does not care
  • rename the name of the different fields so to be more straightforward what they should be used for. For instance the "global" section title is not clear what is "global" (vs "local"?). "Mandatory" is perhaps more accurate but also a bit verbose, maybe we could just put no section name for those and just name the custom section?
  • could the type of the custom attribute(?) be inferred rather than asking the user to choose it? what happens if I wrongly choose string and put "2" for instance ?
@pboettch pboettch added this to the O-2023-03 milestone Dec 13, 2022
@pboettch
Copy link
Collaborator

I'd count 2 days for that because I have the feeling there will be some discussions before during and after for this kind of ergonomic change... 2 days is conservative ;-).

@pboettch
Copy link
Collaborator

pboettch commented Jun 9, 2023

Needs to be discussed what really needs to be done here.

Renaming stuff and making things bold, easy.

Inferring type, well. We cannot change the type of an attribute later on (as of today), so maybe not so good idea.

@jeandet jeandet modified the milestone: O-2023-03 Dec 7, 2023
pboettch added a commit to pboettch/tscat_gui that referenced this issue May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

3 participants