Opened 12 years ago

Closed 12 years ago

Last modified 8 years ago

#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)

215.patch (12.8 KB) - added by erikos 12 years ago.
refine the alert to match the spec

Download all attachments as: .zip

Change History (8)

comment:1 Changed 12 years ago by erikos

  • Description modified (diff)

Changed 12 years ago by erikos

refine the alert to match the spec

comment:2 Changed 12 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 12 years ago by sascha_silbe

  • Cc sascha_silbe added

comment:4 Changed 12 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 12 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 12 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.

comment:7 Changed 8 years ago by godiard

  • Milestone 0.84 deleted

Milestone 0.84 deleted

Note: See TracTickets for help on using tickets.