Skip to content
This repository has been archived by the owner on Nov 11, 2021. It is now read-only.

ORTCDemo: focus state for "connect" button is not obvious enough #196

Open
dstorey opened this issue Dec 8, 2016 · 4 comments
Open

ORTCDemo: focus state for "connect" button is not obvious enough #196

dstorey opened this issue Dec 8, 2016 · 4 comments

Comments

@dstorey
Copy link
Member

dstorey commented Dec 8, 2016

https://developer.microsoft.com/en-us/microsoft-edge/testdrive/demos/ortcdemo/

Changes from transparent background to white, which is very similar. 

It looks like the styles are busted as no top margin (owl removal probably) and the hover state is blue.

@alrra
Copy link
Contributor

alrra commented Dec 16, 2016

@dstorey @sarvaje This demo is external, namely, it's from https://ortcdemo.azurewebsites.net.

@molant
Copy link
Contributor

molant commented Dec 16, 2016

We have access to that server. Don't know where the code is but we can modify the templates directly if needed. Erik should know more and @sarvaje should have access to it via the azure portal.

@sarvaje
Copy link
Contributor

sarvaje commented Dec 16, 2016

Yep, I have access to the server so I can change the css directly. @dstorey is it good just to change the current focus style for the hover style (the blue color)

@dstorey
Copy link
Member Author

dstorey commented Dec 29, 2016

@sarvaje it should probably have something similar to the edgeCore focus styles, as if use same as hover it would be obvious that they're different. Maybe @melanierichards can chime in.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants