Skip to content
This repository has been archived by the owner on Sep 20, 2018. It is now read-only.

Background color for the numbers of 'on.gif' #5

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

SlickHackz
Copy link

After setting the 'ON' icon, when the count (timesAlreadyDone) is less than or equal to 20, then the text appears with 'Green' color background otherwise it is Red

After setting the 'ON' icon, when the count (timesAlreadyDone) is less than or equal to 20, then the text appears with 'Green' color background otherwise it is Red
Screenshots  demonstrating : 
Green - if the Requests count is below 20
Red - if the Requests count is above 20
@awalgarg
Copy link
Owner

Very nice, I like it!

I'd retake the screenshots with my local theme since that gives a bit better effect - so I'd cherry pick your first commit (please keep them separate and do not squash!). Thank you for adding the screenshots though :)

I'd also like to fix #4 first before merging this (will definitely get around to that soon, just busy in work presently).

Thank you for the PR! :)

@SlickHackz
Copy link
Author

Sure thingy... As you say Awal Garg...

Best Regards,
-Gp

On Jul 11, 2016 5:07 PM, "Awal Garg" [email protected] wrote:

Very nice, I like it!

I'd retake the screenshots with my local theme since that gives a bit
better effect - so I'd cherry pick your first commit (please keep them
separate and do not squash!). Thank you for adding the screenshots though :)

I'd also like to fix #4 #4
first before merging this (will definitely get around to that soon, just
busy in work presently).

Thank you for the PR! :)


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#5 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AQ2oFFbWmy7GTUvYdapeeopIJVSBkhZUks5qUisSgaJpZM4JJO0b
.

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

Successfully merging this pull request may close these issues.

2 participants