#215 closed enhancement (fixed)
Refine 'Name this entry' dialog
| Reported by: | erikos | Owned by: | erikos |
|---|---|---|---|
| Priority: | major | Milestone: | |
| Component: | Sugar | Version: | Git as of bugdate |
| Severity: | Minor | Keywords: | r+ |
| Cc: | sascha_silbe, eben | Distribution/OS: | Unspecified |
| Bug Status: | Resolved |
Description (last modified by erikos)
Spec as discussed with Eben:
Header:
[Journal icon] "Name this entry" (...) [X] [√] [X] says "Don't keep entry in Journal", [√] says "Keep it"
Body: Should look like the journal detail view:
[star icon] [activity icon] [title entry]
[description field] [tags field]
optional: display the thumbnail as well
Attachments (1)
Change History (8)
comment:1 Changed 14 years ago by erikos
- Description modified (diff)
Changed 14 years ago by erikos
comment:2 Changed 14 years ago by erikos
- Keywords r? added
- Status changed from new to accepted
a screenshot can be seen here: http://shell.sugarlabs.org/~erikos/naming.png
comment:3 Changed 14 years ago by sascha_silbe
- Cc sascha_silbe added
comment:4 Changed 14 years ago by erikos
- Cc eben added
- Keywords r+ added; r? removed
Got reviewed by tomeu on irc. Made the changes needed and pushed to sugar-toolkit. Moved as well the canvasentrytext to sugar/graphics which I added to jarabe/view last night to avoid circular references.
I want to add the 'enter' key to the quit button. Eben, if you have any other idea, please let me know. Tomeu and myself decided that the don't quit button does not make sense in the dialog, happy to argue again about it.
comment:5 Changed 14 years ago by erikos
'don't keep' button i meant of course. The argument was, that the entry is saved already. And don't keep would mean that the entry needed to be erased. With the added 'resume by default', we don't have the proliferation of useless entries we used to have.
comment:6 Changed 14 years ago by tomeu
- Bug Status set to Resolved
- Distribution/OS set to Unspecified
- Milestone set to 0.84
- Resolution set to fixed
- Severity set to Minor
- Status changed from accepted to closed
Closing this one, please open new tickets about specific issues with this dialog.
refine the alert to match the spec