Keyboard focus is not clearly visible

[lyndaidaii]

Summary of the changes (in less than 80 characters):

Problem: Keyboard focus is not clearly visible because of overflow was set to hidden.

Fix: Changed to visible since the button name length is fix, won't be overflow.

For "Eidt, Regenerate, Delete" buttons, it's ok to

Addresses https://github.com/NuGet/Engineering/issues/4827

[agr]

Could you please post screenshots?

[lyndaidaii]

Could you please post screenshots?

not able to provide with the screenshots since keyboard interaction. video might work. let me upload it.

[agr]

As far as I understand, this

gets changed to this

Given that .package-list class name seems a bit out of place on an API key management page and that we have an explicit overflow-y: hidden for .package-list li selector

NuGetGallery/src/Bootstrap/less/theme/base.less

Line 367 in 0931f8b

overflow-y: hidden;

I would guess that combination of elements is used elsewhere, too. I wonder if the focus indication is broken there, too.

[agr]

OK, it is on package list page and respective elements there are not focusable.