Merge lp:~rockstar/phazr/images into lp:phazr
Proposed by
Paul Hummer
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Paul Hummer | ||||
Approved revision: | 11 | ||||
Merged at revision: | 5 | ||||
Proposed branch: | lp:~rockstar/phazr/images | ||||
Merge into: | lp:phazr | ||||
Prerequisite: | lp:~rockstar/phazr/close-overlay | ||||
Diff against target: |
65 lines (+19/-4) 3 files modified
src/css/phazr.css (+16/-1) src/js/formplugin/formplugin.js (+2/-2) src/js/modalplugin/modalplugin.js (+1/-1) |
||||
To merge this branch: | bzr merge lp:~rockstar/phazr/images | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Deryck Hodge | code | Approve | |
Review via email: mp+54460@code.launchpad.net |
Commit message
Add images for okay, cancel, and close
Description of the change
This branch adds the images and css for Okay, Cancel, and Close. These images are a bit dark, and probably need some work (they didn't shrink from their original 128x128 size very well). I wanted to stick them in, and if we have time to come back and fix them before release, we can do that.
To post a comment you must log in.
I'm ok with this. I agree that the images could be touched up perhaps. And there's some weird spacing between the cancel and save icons when sat beside each other. But as a first pass to cleanup later, I think it's fine.
At some point, we should add docs about how to easily re-skin or use different images. Images are such a personal issue. I like these, but not every project will want them.
Cheers,
deryck