toolbar; honour admin css styles (colors) #55

Closed
futtta opened this Issue Aug 3, 2016 · 3 comments

Comments

2 participants
@futtta
Owner

futtta commented Aug 3, 2016

got this remark from a tester;

The Red icon does not show on the Sunrise admin color scheme. In general, the color of the icon should be aware of the admin color scheme so it can contrast.

if you have any idea how to do that @pablocusto , "go"! ;-)

But I consider this minor, really (the timeout is way more important).

@futtta futtta added the enhancement label Aug 3, 2016

@pablocusto

This comment has been minimized.

Show comment
Hide comment
@pablocusto

pablocusto Aug 3, 2016

Contributor

Oops! LOL.. added to next fix.

in a couple of hours I make a new pull request with the fix of the timeout
issue with your suggestion of the modal popup

El 3/8/2016 3:32, "frank goossens" notifications@github.com escribió:

got this remark from a tester;

The Red icon does not show on the Sunrise admin color scheme. In general,
the color of the icon should be aware of the admin color scheme so it can
contrast.

if you have any idea how to do that @pablocusto
https://github.com/pablocusto , "go"! ;-)

But I consider this minor, really (the timeout is way more important).


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#55, or mute the thread
https://github.com/notifications/unsubscribe-auth/AArPdf_ALrCgfqmAIaOPvQC8jyU9kPrZks5qcDX5gaJpZM4JbWMb
.

Contributor

pablocusto commented Aug 3, 2016

Oops! LOL.. added to next fix.

in a couple of hours I make a new pull request with the fix of the timeout
issue with your suggestion of the modal popup

El 3/8/2016 3:32, "frank goossens" notifications@github.com escribió:

got this remark from a tester;

The Red icon does not show on the Sunrise admin color scheme. In general,
the color of the icon should be aware of the admin color scheme so it can
contrast.

if you have any idea how to do that @pablocusto
https://github.com/pablocusto , "go"! ;-)

But I consider this minor, really (the timeout is way more important).


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#55, or mute the thread
https://github.com/notifications/unsubscribe-auth/AArPdf_ALrCgfqmAIaOPvQC8jyU9kPrZks5qcDX5gaJpZM4JbWMb
.

@futtta

This comment has been minimized.

Show comment
Hide comment
@futtta

futtta Aug 3, 2016

Owner

with your suggestion of the modal popup

great, but I meant a wordpress notice, in the page underneath the h1 :-)

Owner

futtta commented Aug 3, 2016

with your suggestion of the modal popup

great, but I meant a wordpress notice, in the page underneath the h1 :-)

@pablocusto

This comment has been minimized.

Show comment
Hide comment
@pablocusto

pablocusto Aug 3, 2016

Contributor

Oki.. nice idea. :-)

El 3/8/2016 16:28, "frank goossens" notifications@github.com escribió:

with your suggestion of the modal popup

great, but I meant a wordpress notice, in the page underneath the h1 :-)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#55 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArPdchKC9RXfnx9ixsxcbC5PaAgpAzyks5qcOvhgaJpZM4JbWMb
.

Contributor

pablocusto commented Aug 3, 2016

Oki.. nice idea. :-)

El 3/8/2016 16:28, "frank goossens" notifications@github.com escribió:

with your suggestion of the modal popup

great, but I meant a wordpress notice, in the page underneath the h1 :-)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#55 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArPdchKC9RXfnx9ixsxcbC5PaAgpAzyks5qcOvhgaJpZM4JbWMb
.

@futtta futtta closed this Jun 25, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment