Lee Moody comments

Results 79 comments of


                                            Lee Moody

o-tooltip in hover mode has a cross to close it

Thanks for flagging, that's definitely a boog. We'd like to audit tooltip to fix accessibility issues and add usage guidelines - this would be a good one to clean up...

o-layout's sidebar should indicate hierarchy of headings

2 levels of hierarchy are indicated by the change in the right boarder colour / width, but it's not obvious at all and should be improved ![Screenshot 2021-10-11 at 09...

o-layout's sidebar should indicate hierarchy of headings

that could work!

Make the display-after option compatible with display on focus/hover

A small delay on hover/focus could be a good idea, we should raise this with design when we review o-tooltip (right now there are lots of options, not good guidance,...

Expose current release number to the component's own code

There are some trade offs in terms of workflow and reliably updating the version before tagging, but I think this would be useful 👍 particularly for npm users who do...

Expose current release number to the component's own code

looks like [o-tracking uses its version number too](https://github.com/Financial-Times/o-tracking/issues/223)

feat: introduce `o3-grid-column-width` CSS Custom Property

Looks good 😄 Now just to document in the README and potentially add a demo, showing a nested element aligned to the grid, to the Story?

chore: upgrade ftdomdelegate to v5 in o-table

@ob6160 please update the commit message to `fix:` so this is released on merge

fix: Prevent an error if no sticky search button is used.

@j-mes Yeah, in that it's more resilient, but no one is waiting on it. Thanks for the reminder. I'm probably not going to get to this before leaving so feel...