search

patternfly / chatbot

This repo contains the code for the PatternFly virtual assistant extension
MIT License
0 stars 21 forks source link

Add focus outline to chatbot #305

Closed rebeccaalpert closed 2 weeks ago

rebeccaalpert commented 2 weeks ago

We should make sure that there's a focus outline in fullscreen and default and embedded. (Docked is fine.) This seems tied to the child components - I wonder if we can refactor a little so this won't be an issue.

Culprit seems to be the MessageBox without the history drawer, specifically 

.pf-chatbot__content {
    position: relative; }

position: relative; on the header also seems like a problem child.

Drawer is also a problem child.

See https://github.com/patternfly/virtual-assistant/pull/296#pullrequestreview-2421031229.

rebeccaalpert commented 2 weeks ago

Fixed as part of initial PR - took a break and a solution came to me. :)