#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: | ||
| 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)
Change History (9)
by , 18 years ago
| Attachment: | Patch set.zip added |
|---|
comment:1 by , 18 years ago
comment:2 by , 18 years ago
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 by , 18 years ago
| Status: | new → assigned |
|---|
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 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
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.
by , 18 years ago
| Attachment: | Magnify.cpp.diff added |
|---|
comment:6 by , 18 years ago
I seem to have missed to include the patch for Magnify. Sorry about that. See attached file Magnify.cpp.diff
Let me know if you want the patches uploaded individually, or grouped somehow.