Hi Gary,

thanks for having a look!

On 08/21/2012 02:34 PM, Gary Martin wrote:
Hi Simon,

On 21 Aug 2012, at 12:18, Simon Schampijer <[email protected]> wrote:

Hi,

this Feature "Display Device" [1] has been proposed for 0.96 inclusion already 
but did not make it in due to time constraints. With touch the need to expose the 
'take-screenshot' and 'adjust-brightness' options in the UI gets more prominent, that is 
why I want to propose this again for 0.98.

The part for the screenshot taking was already done, that part of the patch 
just needs to be re-based.

I'm not happy the design is ready yet for inclusion, some quick comments:

  - Use of the camera icon (especially in the device frame) suggests it 
involves camera hardware.

The icon is a placeholder. It should be an icon for the display.

  - Is taking a screenshot really a feature that should be in the device 
section of the frame?

See below. I am happy to hear about other suggestions. At the moment I have no other one :/ Previous discussions did not bring something up neither [1].

  - The device frame is filling up fast and furious in this cycle, we already 
need to add an icon for raising the OSK, and an icon for controlling the 
hardware display brightness/monochrome mode.
  - The palette shown in the screen shot is named Display.

The Palette should contain as well the options for the brightness. At least that is what we discussed at one point. The Palette would be named something like 'Display', have an option for brightness and one for taking a screenshot.

Regards,
   Simon

[1] http://lists.sugarlabs.org/archive/sugar-devel/2011-November/034372.html


_______________________________________________
Sugar-devel mailing list
[email protected]
http://lists.sugarlabs.org/listinfo/sugar-devel

Reply via email to