Skip to content
This repository was archived by the owner on Mar 15, 2018. It is now read-only.

change close button in active state (bug 979122)#528

Merged
dethe merged 1 commit intomozilla:masterfrom
dethe:fix_active_modal_button
Jul 23, 2014
Merged

change close button in active state (bug 979122)#528
dethe merged 1 commit intomozilla:masterfrom
dethe:fix_active_modal_button

Conversation

@dethe
Copy link
Copy Markdown
Contributor

@dethe dethe commented Jul 23, 2014

Not sure why the existing close_active.svg was not being added as a background to replace close.svg in active state, but this adds it.

Comment thread src/media/css/header.styl
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like this should be a sprite but it's out of scope for this commit.

@spasovski
Copy link
Copy Markdown
Contributor

arrrr +

@dethe
Copy link
Copy Markdown
Contributor Author

dethe commented Jul 23, 2014

@spasovski +1 for pirate plus.

dethe added a commit that referenced this pull request Jul 23, 2014
change close button in active state (bug 979122)
@dethe dethe merged commit 41fd7cc into mozilla:master Jul 23, 2014
@dethe dethe deleted the fix_active_modal_button branch July 23, 2014 23:28
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