Feature/finished/iia 1335 numbered editable list#280
Conversation
…xing bugs of Editable List
# Conflicts: # application/pom.xml # artifact/pom.xml # builder/pom.xml # classification/pom.xml # details/pom.xml # executor/pom.xml # framework/pom.xml # knowledge-layout/pom.xml # komet-terms/pom.xml # kview/pom.xml # list/pom.xml # navigator/pom.xml # pom.xml # preferences/pom.xml # progress/pom.xml # rules/pom.xml # search/pom.xml
|
@dukke Pedro, Let's confirm this together & design team. I was thinking of two option for this PR:
Notes:
Outstanding questions:
|
|
@carldea Thanks for the detailed comments to this PR. Notes:1 - I think I understand what you're saying but there are practical reasons to why that doesn't happen on this PR. We don't have space between the title and the first component to draw a drop line there. We can add more space between the title and the first component but then we'll have to do that for every editable component for consistency's sake. Perhaps we could do that, but we might want to add another ticket as this one is already getting too big (it's already fixing 3 issues) and contains tickets there were target for the previous sprint. 2 - Not sure I understand this one but maybe we can talk in a video meeting... Outstanding questions:Those are good questions but maybe we can tackle them outside of this PR as this PR is already too big and also references tickets that were already in a previous sprint? So perhaps we finish or merge this (if we agree on how things are implemented and done) and then we can tackle other issues on a different ticket? |
|
I agree it is getting large (taking too long) and changes in the Figma spec happened after your merge was created. At least we have a working version. 👍🏻 |
|
@carldea thanks for your valuable feedback. Just merged. I'm going to create tickets for your other comments so we can address those issues next. |




Fixes:
Video capture of how it's working with this PR:
2025-02-21.15-39-03.mp4