put the search icon in the headerbar
Bug #1360529 reported by
Danielle Foré
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Photos |
Fix Released
|
Medium
|
meese | ||
Bug Description
Let's put the search icon up in the top-right in the headerbar so it's like our other apps.
Related branches
lp:~meese/pantheon-photos/searchbutton-header
- Viko Adi Rahmawan (community): Needs Fixing
- Danielle Foré: Approve (ux)
- elementary Apps team: Pending (code) requested
-
Diff: 290 lines (+86/-19)9 files modifiedsrc/Page.vala (+13/-3)
src/PhotoPage.vala (+23/-0)
src/camera/ImportPage.vala (+0/-6)
src/events/EventPage.vala (+23/-1)
src/library/Branch.vala (+4/-0)
src/library/LibraryWindow.vala (+23/-6)
ui/collection.ui (+0/-1)
ui/offline.ui (+0/-1)
ui/trash.ui (+0/-1)
Revision history for this message
| Danielle Foré (danrabbit) wrote | #1 |
| Changed in pantheon-photos: | |
| status: | New → Invalid |
| status: | Invalid → Won't Fix |
Revision history for this message
| Danielle Foré (danrabbit) wrote | #2 |
| Changed in pantheon-photos: | |
| status: | Won't Fix → Confirmed |
| description: | updated |
| summary: |
- put the search icon in the titlebar + put the search icon in the headerbar |
| Changed in pantheon-photos: | |
| status: | Confirmed → In Progress |
| assignee: | nobody → meese (meese) |
| Changed in pantheon-photos: | |
| status: | In Progress → Fix Committed |
| Changed in pantheon-photos: | |
| importance: | Undecided → Medium |
| milestone: | none → freya-beta2 |
| Changed in pantheon-photos: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
nvm looks stupid.