futtta / ao_critcss_aas

Autoptimize power-up to integrate with criticalcss.com
9 stars 1 forks source link

jobs status badges improvements? #42

Closed futtta closed 6 years ago

futtta commented 6 years ago

as per Jonas' comment;

pocketjoso commented 6 years ago

After some time the default HTML tooltip shows up. Is this what is meant to happen? tooltip-missing

futtta commented 6 years ago

yeah, that's what it should do indeed.

On Sun, Jul 8, 2018 at 7:38 PM, Jonas Ohlsson Aden <notifications@github.com

wrote:

After some time the default HTML tooltip shows up. Is this what is meant to happen? [image: tooltip-missing] https://user-images.githubusercontent.com/7192832/42422444-4f9541b8-82e6-11e8-88b8-45b6b67efcf0.gif

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/futtta/ao_critcss_aas/issues/42#issuecomment-403303510, or mute the thread https://github.com/notifications/unsubscribe-auth/AALEMcfbdi61erIXpsV8leSGXRlZuBmmks5uEkN8gaJpZM4VGw-g .

pocketjoso commented 6 years ago

okay, default html tooltips always bother me because they are slow, but not really a bug there then. Just the fact that the element has a click cursor, while clicking does nothing.

futtta commented 6 years ago

we could switch the click cursor to an arrow?

On Sun, Jul 8, 2018 at 8:40 PM, Jonas Ohlsson Aden <notifications@github.com

wrote:

okay, default html tooltips always bother me because they are slow, but not really a bug there then. Just the fact that the element has a click cursor, while clicking does nothing.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/futtta/ao_critcss_aas/issues/42#issuecomment-403307246, or mute the thread https://github.com/notifications/unsubscribe-auth/AALEMZM12MXQSVuOJnanS-mfm92ioXMQks5uElIGgaJpZM4VGw-g .

pocketjoso commented 6 years ago

Yes, better fit here.

denydias commented 6 years ago

A simple arrow does not indicate there's something more to be seen. CSS3 context-menu or help would be a better fit. A list of available CSS cursors is available here. Please indicate one or confirm default (arrow).

Also default HTML tooltips doesn't require any extra JS for displaying them. WP already has enough JS IMHO.

I also had to go hunting to understand what N meant

RTFM will do.

ao_critcss_aas

futtta commented 6 years ago

we'll go "default" :-)

On Thu, Jul 12, 2018 at 7:20 AM, Deny Dias notifications@github.com wrote:

A simple arrow does not indicate there's something more to be seen. CSS3 context-menu or help would be a better fit. A list of available CSS cursors is available here https://codepen.io/benlinton/full/JXzZBp/. Please indicate one or confirm default (arrow).

Also default HTML tooltips doesn't require any extra JS for displaying them. WP already has enough JS IMHO.

I also had to go hunting to understand what N meant

RTFM will do.

[image: ao_critcss_aas] https://user-images.githubusercontent.com/948202/42613828-4edc44b6-8579-11e8-9846-f4d1d48ee295.png

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/futtta/ao_critcss_aas/issues/42#issuecomment-404392567, or mute the thread https://github.com/notifications/unsubscribe-auth/AALEMSIKjvTv2_14pqhSUVKL2PA4vY_sks5uFtyqgaJpZM4VGw-g .