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: Sub-Categories for config-file / Grading-Tab #77

Open
Shirkanesi opened this issue Apr 20, 2022 · 4 comments
Open

Feature: Sub-Categories for config-file / Grading-Tab #77

Shirkanesi opened this issue Apr 20, 2022 · 4 comments

Comments

@Shirkanesi
Copy link

Is your feature request related to a problem? Please describe.
Currently there is only an option to create different categories for mistake-types that can internally be capped to specific limits.
Therefore using those categories to sort the buttons in the UI is not possible (without losing the penalty-cap).
As the location of buttons will (most likely) change with a change in the config-file, grading quickly gets quite frustrating (as our monkey-brains will initialy memorize the location of certain buttons rather quickly and adapting to a change in the layout takes quite long).

Describe the solution you'd like
There should be an option to additionally group penalty-buttons by logical categories (as "comments", "language", ...) to enforce logical sorting of the buttons and to minimize changes in the layout (as an addition of one button regarding e.g. "comments" only affects relative positions inside this sub-category and all other categories will stay untouched.
In general finding specific buttons will be sped up quite a lot when similar buttons are next to each other.


Note: For implementation it should be ensured those changed won't create a breaking change in created feedback but only in the config-files.
(I don't see a reason why this should happen hence only the "shortName" is used for the Feedback-JSON)

@Shirkanesi
Copy link
Author

In this process, deserializing the configuration-files could also be cleaned up a bit.

@dfuchss
Copy link
Member

dfuchss commented May 7, 2022

Currently, I'm not working on this :)

@Shirkanesi
Copy link
Author

We need to discuss about this feature in the future as it is a breaking change in the config-file, so the way this is actually done should be made clear before developing the feature.

@dfuchss
Copy link
Member

dfuchss commented Oct 13, 2022

Is planned to be part of a practical course task this term

@dfuchss dfuchss transferred this issue from kit-sdq/eclipse-artemis Jul 14, 2024
@dfuchss dfuchss moved this to Triage in Artemis & Programming Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

2 participants