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

Cannot edit data class attributes #147

Open
LeonBein opened this issue Jul 19, 2022 · 1 comment
Open

Cannot edit data class attributes #147

LeonBein opened this issue Jul 19, 2022 · 1 comment

Comments

@LeonBein
Copy link
Collaborator

When
In the data modeler, I edit data class attributes and confirm changes with enter or left-click-outside

I expected
That the entered attributes would be permanently displayed in the data class

But actually
The data class stayed blank

@LeonBein LeonBein added bug Something isn't working questionable behavior and removed bug Something isn't working labels Jul 19, 2022
@LeonBein
Copy link
Collaborator Author

LeonBein commented Jul 19, 2022

This is no bug, as the intended behavior is to not accept text that does not conform "attributeName : dataType", where dataType is one of a predefined set.
Cp.

var check_re_class = /[A-Za-z]+: string|String|integer|Integer|int|Int|Str|str|Float|float|Boolean|boolean|bool|Bool$/
if (!text.match(check_re_class)) {
text = oldText;

However, this behavior is unintuitive in that there is no indicator to the user on why text was not accepted; if there is no knowledge about this code line, users will assume it to be a bug.

AnjoSs pushed a commit that referenced this issue Jul 27, 2023
* * Adds a Dropdown that contains Name,Duration,Role and Number of People

* * Fixes a small Problem with the DatObjectLabelHandler

* Working on a CustomRenderer

* Working on a CustomRenderer pt.2

* * Implements the Custom Acticity Renderer

* * Enables Import/Export
* Fixes a small bug where direct editing of the labels was enabled

* Changing the durations location

* Renaming CustomRenderer to TaskRenderer

* * Deletes fragmentLabelHandler

---------

Co-authored-by: Noel <[email protected]>
Co-authored-by: lottchen <[email protected]>
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

1 participant