Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1212 closed bug (fixed)

human interface guideline patches: ellipsis and related in menu items, buttons

Reported by: jonas.kirilla Owned by: axeld
Priority: normal Milestone: R1
Component: User Interface Version: R1/pre-alpha1
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

Patches to apps, prefs, BDragger and libprint.

Apps: Cortex, DeskCalc, Expander, Mail, MediaPlayer, NetworkStatus, People, Poorman, ProcessController, Pulse, StyledEdit, Terminal, TV

Prefs: DUN, DataTranslations, Devices, Menu, Network

Interface kit: Dragger.cpp

Misc: JobSetupDlg.cpp

Attachments (2)

Patch set.zip (18.7 KB) - added by jonas.kirilla 12 years ago.
Magnify.cpp.diff (648 bytes) - added by jonas.kirilla 12 years ago.

Download all attachments as: .zip

Change History (9)

Changed 12 years ago by jonas.kirilla

Attachment: Patch set.zip added

comment:1 in reply to:  description Changed 12 years ago by jonas.kirilla

Let me know if you want the patches uploaded individually, or grouped somehow.

comment:2 Changed 12 years ago by jonas.kirilla

I should probably mention that I've changed Terminal's menu item "Start New Terminal" to say just "New Terminal…" to be more in line with the other applications. This deviates from R5 but I think it makes sense.

comment:3 Changed 12 years ago by axeld

Status: newassigned

Thanks for the patches, I'll look into applying them soon. But to answer your question above, it would have been nice to only have a single patch to apply in this case :-)

comment:4 Changed 12 years ago by axeld

Resolution: fixed
Status: assignedclosed

Thanks, I've applied it in hrev21156, including the Terminal changes. I've also added missing ellipsis in Poorman's "About Poorman" and the Deskbar.

comment:5 Changed 12 years ago by jonas.kirilla

Thanks!

Changed 12 years ago by jonas.kirilla

Attachment: Magnify.cpp.diff added

comment:6 Changed 12 years ago by jonas.kirilla

I seem to have missed to include the patch for Magnify. Sorry about that. See attached file Magnify.cpp.diff

comment:7 Changed 12 years ago by axeld

No problem, I've applied it in hrev21227.

Note: See TracTickets for help on using tickets.