[haiku-bugs] Re: [Haiku] #7002: Context menu for TextInput

  • From: "dancxjo" <trac@xxxxxxxxxxxx>
  • Date: Thu, 23 Dec 2010 18:03:05 -0000

#7002: Context menu for TextInput
--------------------------+------------------------------------------------
  Reporter:  dancxjo      |        Owner:  axeld
      Type:  enhancement  |       Status:  new
  Priority:  normal       |    Milestone:  R1
 Component:               |      Version:  R1/Development
  Kits/Interface Kit      |     Keywords:  context menu text edit textinput
Resolution:               |  Has a Patch:  1
Blocked By:               |     Blocking:
  Platform:  All          |
--------------------------+------------------------------------------------

Comment (by dancxjo):

 >  - Uncommon abbreviations for identifier names should be avoided, i.e.
 e.g. `undoStr` should be renamed to `undoString`/`undoName`/`undoLabel`.
 Removed altogether; but is the macro okay? The name is perhaps too generic
 and it assumes that the Locale Backend isn't null. Is this a bad idea???

 >  - `B_UNDO` is erroneously used for the redo menu item.
 There is no such thing as a B_REDO, AFAIK. Redo is just undo when the last
 action was an undo. I'm not very pleased with it, but I didn't want to
 hack the undo buffer.

 Thank you, bonefish, for the advice so far!

-- 
Ticket URL: <http://dev.haiku-os.org/ticket/7002#comment:13>
Haiku <http://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: