iTop
iTop copied to clipboard
N°8893 - :lipstick: Transition validation markup
Base information
| Question | Answer |
|---|---|
| Related to a SourceForge thead / Another PR / Combodo ticket? | No |
| Type of change? | Enhancement |
Symptom (bug) / Objective (enhancement)
When using for example the new event listeners to perform a check before the object is written during a transition; the current error modal is ugly. It does not use a nice list; and it shows context data.
Proposed solution (bug and enhancement)
The error messages are just examples.
Before:
After:
Checklist before requesting a review
- [x] I have performed a self-review of my code
- [x] I have tested all changes I made on an iTop instance
- [ ] I have added a unit test, otherwise I have explained why I couldn't
- [x] Is the PR clear and detailed enough so anyone can understand digging in the code?
Checklist of things to do before PR is ready to merge
Accepted during functional review. Will check with @rquetiez for technical details (3.2 compatibility and way to define the message).