[touch] The brightness/volume changes are not realtime

Bug #1227595 reported by Omer Akram
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
unity8 (Ubuntu)
Fix Released
Low
Omer Akram

Bug Description

Changing the screen brightness on my touch devices is not real time. The brightness should changes as we move along the slider but it seems the screen brightness is only changed when the the slider is untapped.

1. open battey Menu
2. try to change the brightness of the screen

What happens:
The brightness only changes when you leave the slider

What should happen:
Brightness change should be live.

ProblemType: Bug
DistroRelease: Ubuntu 13.10
Package: indicator-power 12.10.6+13.10.20130913-0ubuntu2
Uname: Linux 3.4.0-3-mako armv7l
ApportVersion: 2.12.2-0ubuntu1
Architecture: armhf
Date: Thu Sep 19 11:27:57 2013
InstallationDate: Installed on 2013-09-18 (0 days ago)
InstallationMedia: Ubuntu Saucy Salamander (development branch) - armhf (20130918)
MarkForUpload: True
ProcEnviron:
 TERM=linux
 PATH=(custom, no user)
SourcePackage: indicator-power
UpgradeStatus: No upgrade log present (probably fresh install)

Related branches

Revision history for this message
Omer Akram (om26er) wrote :
Changed in indicator-power:
importance: Undecided → Low
Changed in indicator-power (Ubuntu):
importance: Undecided → Low
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in indicator-power (Ubuntu):
status: New → Confirmed
Charles Kerr (charlesk)
Changed in indicator-power:
status: New → Invalid
Changed in unity8:
status: New → Confirmed
Revision history for this message
Charles Kerr (charlesk) wrote :

This is coming from unity8/plugins/Unity/Indicators/qml/SliderMenuItem.qml, which uses the Ubuntu UI Toolkit's slider but explicitly sets the 'live' property to false so that there won't be realtime updates.

I'm not sure whether or not there was a Good Reason it was done this way. Looks like Renato wrote the original version that included the "live: false" line, so assigning him under also-affects Unity 8 so that he can give a second opinion on this ticket.

Changed in unity8:
importance: Undecided → Low
Omer Akram (om26er)
no longer affects: indicator-power
affects: indicator-power (Ubuntu) → unity8 (Ubuntu)
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

@charles

I think I did that way because I was having problems to keep sync btw the UI and the backend, since the backend notifies the UI every time that the device changes the property, sometimes moving the slider is faster than update the device property, this cause the backend to notify a old value and the UI was changing to a old value instead of keep sync with the user interaction.

In my opinion the correct is ignore the backend updates during the user interaction but since the SDK component does not provide a way to know that (during the time that I was working on that), the easy way was update after the user interaction.

Michał Sawicz (saviq)
Changed in unity8:
status: Confirmed → Triaged
assignee: nobody → Omer Akram (om26er)
status: Triaged → In Progress
Omer Akram (om26er)
summary: - [touch] The screen brightness change is not realtime
+ [touch] The screen brightness/volume change is not realtime
summary: - [touch] The screen brightness/volume change is not realtime
+ [touch] The brightness/volume changes are not realtime
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:unity8 at revision 382, scheduled for release in unity8, milestone ubuntu-13.09

Changed in unity8:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.2 KiB)

This bug was fixed in the package unity8 - 7.82+13.10.20131004.1-0ubuntu1

---------------
unity8 (7.82+13.10.20131004.1-0ubuntu1) saucy; urgency=low

  [ Michał Sawicz ]
  * Bump to indicate support for extended snap decisions.
  * Make Tile themeable, add renderers for Dash Plugins and weather.
    (LP: #1231948)
  * Drop network agents now that they're in indicator-network instead.

  [ Michal Hruby ]
  * Differentiate generic and music carousels.
  * Use the thumbnailer image provider for scope results that don't
    specify icon as well as for previews.
  * Added an indicator which is displayed in the search bar whenever a
    search is in progress. Added accompanying test in tst_PageHeader.
  * Expose rendererHint to shell.

  [ Albert Astals ]
  * Update pot file. (LP: #1232374)
  * Only enable the animation when the item is on screen (i.e. !culled)
    . (LP: #1200374)
  * Do not crash on positionAtBeginning if the list is empty .
  * Enable/disable running apps height animation in a less error prone
    way.

  [ Michael Terry ]
  * Use a chevron after 'Skip intro' and drop the underlining.
  * Load testability driver when QT_LOAD_TESTABILITY is set. (LP:
    #1226234)
  * Listen to the system setting StatsWelcomeScreen, which tells us
    whether to show user-specific infographic data in the greeter. (LP:
    #1207857)

  [ Gerry Boland ]
  * WM: ensure focusedApplicationWhenUsingScreenshots reset when unused,
    and only used when set. Fixes window focus conflict between shell
    and ApplicationManager.

  [ Nick Dedekind ]
  * Removed deprecated Unity.IndicatorsLegacy plugin.
  * Added actionState parser to the indicators-client text printer so
    that we get the icon.
  * Moved indicator page titles to the root action state of menu model.
    (LP: #1223635)

  [ Mirco Müller ]
  * Added rendering- and interaction-support for the first three
    extended snap-decision dialog-cases password-entry, user-
    authentication and simunlock.

  [ Daniel d'Andrada ]
  * Revert the reversion of r304 since it doesn't seem to crash anymore
    Original commit was Reset apps scope when returning from app to dash
    (LP #1193419) If an app is on foreground and you perform a long
    left-edge swipe to minimize it, and therefore return to the dash,
    the dash should be in the Applications scope and showing the
    running/recents applications. (LP: #1193419)

  [ Andrea Cimitan ]
  * Add behaviours to the hud reveal. (LP: #1224480, #1224633)
  * Make Tile themeable, add renderers for Dash Plugins and weather.
    (LP: #1231948)

  [ Michael Zanetti ]
  * fix inserting into quicklistmodel.
  * Drop useStorage argument and use exising LAUNCHER_TESTING define for
    this decision.

  [ Omer Akram ]
  * Make the volume/brightness slider changes realtime. (LP: #1227595)

  [ Nicolas d'Offay ]
  * Fixed black colour on the first of the month due to division. (LP:
    #1233657)
  * Added an indicator which is displayed in the search bar whenever a
    search is in progress. Added accompanying test in tst_PageHeader.

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 390
 -- Ubuntu daily release <email address hidden> ...

Read more...

Changed in unity8 (Ubuntu):
status: Confirmed → Fix Released
Michał Sawicz (saviq)
Changed in unity8:
status: Fix Committed → Fix Released
tags: added: doesnt-need-test
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu):
assignee: nobody → Omer Akram (om26er)
no longer affects: unity8
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.