Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GENERIC viewer] Fix the background color of the findInput when the search term is not found #5465

Merged
merged 1 commit into from
Nov 18, 2014
Merged

[GENERIC viewer] Fix the background color of the findInput when the search term is not found #5465

merged 1 commit into from
Nov 18, 2014

Conversation

Snuffleupagus
Copy link
Collaborator

When a search term isn't found, the background color of the findInput is supposed to change (to red). This is currently not working as intended, because the CSS rule is not being applied correctly. (It seems that this broke in PR #2208.)

This patch also changes the background color to match the one used in the native Firefox findbar, since the old color seemed a bit too pink.

…earch term is not found

When a search term isn't found, the background color of the findInput is supposed to change (to red). This is currently not working as intended, because the CSS rule is not being applied correctly. (It seems that this broke in PR 2208.)

This patch also changes the background color to match the one used in the native Firefox findbar, since the old color seemed a bit too pink.
@Snuffleupagus
Copy link
Collaborator Author

/botio-windows preview

@pdfjsbot
Copy link

pdfjsbot commented Nov 2, 2014

From: Bot.io (Windows)


Received

Command cmd_preview from @Snuffleupagus received. Current queue size: 0

Live output at: http://107.22.172.223:8877/bb8c8eb0d4a88e0/output.txt

@Snuffleupagus
Copy link
Collaborator Author

@timvandermeij Do you have time to review this? (If not, just unassign yourself ;-)

timvandermeij added a commit that referenced this pull request Nov 18, 2014
[GENERIC viewer] Fix the background color of the findInput when the search term is not found
@timvandermeij timvandermeij merged commit dc5961d into mozilla:master Nov 18, 2014
@timvandermeij
Copy link
Contributor

Looks good, thanks!

@Snuffleupagus Snuffleupagus deleted the findbar-notFound-color branch November 18, 2014 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants