bpmn-io / properties-panel

Library for creating bpmn-io properties panels.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Improve the semantics of the properties panel header

rpkoller opened this issue · comments

Is your feature request related to a problem? Please describe

At the moment the header of the properties panel is just using divs for the markup. in Camunda the title of the model is marked up with an h3 in the properties panel but the section underneath with start event and the beginning is just wrapped in divs:
Screen Shot 2023-01-15 at 16 34 06
In ECA in Drupal which doesn't have an h3 for the model title in the properties panel there are just the divs:
Screen Shot 2023-01-15 at 16 35 03

Describe the solution you'd like
Would it make sense to change the html element for the div with the class bio-properties-panel-header to header. And change the div with the class bio-properties-panel-header-type to a heading element. the only question how to handle the sub heading the beginning in the screenshots with the class bio-properties-panel-header-label. wrap it in another heading or wrap both in a single heading and use for one of the them something like thestrong tag. like option 4 in the following article https://css-tricks.com/html-for-subheadings-and-headings/

Describe alternatives you've considered

Additional context

Thank you for your suggestions. Indeed it will make more sense if we use semantic HTML in the properties panel.