[Bugs] #2006 UNSP: add touchpad section to Sugar CP

Sugar Labs Bugs bugtracker-noreply at sugarlabs.org
Mon Aug 16 12:11:12 EDT 2010


#2006: add touchpad section to Sugar CP
------------------------------------------+---------------------------------
    Reporter:  walter                     |          Owner:  tomeu      
        Type:  enhancement                |         Status:  new        
    Priority:  Unspecified by Maintainer  |      Milestone:  0.90       
   Component:  sugar                      |        Version:  Unspecified
    Severity:  Unspecified                |       Keywords:  r+         
Distribution:  OLPC                       |   Status_field:  Unconfirmed
------------------------------------------+---------------------------------
Changes (by tomeu):

  * keywords:  r? => r+


Comment:

 > +               TOUCHPAD_MODES[1]: _('stylus')}

 I guess s/TOUCHPAD_MODES[1]/TOUCHPAD_MODE_RESISTIVE ?

 > +        """ Toggle the touchpad mode; called by touchpad icon
 > +        button-release-event handler. """

 I think we shouldn't be so specific about who calls what because otherwise
 we are going to end up with unsync'ed docstrings, specially for public
 methods.

 > +        # Set the initial touchpad device value to capacitive mode
 (finger)
 > +        _write_touchpad_mode(TOUCHPAD_MODE_CAPACITIVE)

 Isn't this comment redundant? Once you get the intent of your code to be
 clear, you don't need inline comments any more ;)

 Please push with this changes, thanks!

-- 
Ticket URL: <http://bugs.sugarlabs.org/ticket/2006#comment:35>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system


More information about the Bugs mailing list