> On July 7, 2014, 11:57 p.m., David Edmundson wrote: > > src/declarativeimports/plasmacomponents/qml/Slider.qml, line 143 > > <https://git.reviewboard.kde.org/r/119163/diff/2/?file=288502#file288502line143> > > > > This needs bounding to max/min. > > > > i.e if step size is 10 and we press up when we're at 95 we don't want > > to go over
EDIT: just noticed that you've just moved this, that's not a regression. Ship it! - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119163/#review61868 ----------------------------------------------------------- On July 7, 2014, 6:51 p.m., Frederik Gladhorn wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119163/ > ----------------------------------------------------------- > > (Updated July 7, 2014, 6:51 p.m.) > > > Review request for Plasma, Sebastian Sauer and Jeremy Whiting. > > > Repository: plasma-framework > > > Description > ------- > > Adds some accessibility properties to the plasma-framework controls. More > work is needed, but these make many elements show up in accessibility helper > tools. > > > Diffs > ----- > > src/declarativeimports/plasmacomponents/qml/TabButton.qml 21302ef > src/declarativeimports/plasmacomponents/qml/TextArea.qml 0ca6fe5 > src/declarativeimports/plasmacomponents/qml/TextField.qml d67dda3 > src/declarativeimports/plasmacomponents/qml/ToolBar.qml 91b8f77 > src/declarativeimports/plasmacomponents/qml/ToolButton.qml 21afc21 > src/declarativeimports/plasmacomponents/qml/private/DualStateButton.qml > 8c6919e > src/declarativeimports/plasmacomponents/qml/Button.qml 186a39a > src/declarativeimports/plasmacomponents/qml/CommonDialog.qml 75f0362 > src/declarativeimports/plasmacomponents/qml/Label.qml 033c41c > src/declarativeimports/plasmacomponents/qml/ListItem.qml 4e6d5e7 > src/declarativeimports/plasmacomponents/qml/ModelContextMenu.qml 74d81d1 > src/declarativeimports/plasmacomponents/qml/ProgressBar.qml b85c100 > src/declarativeimports/plasmacomponents/qml/RadioButton.qml 769682b > src/declarativeimports/plasmacomponents/qml/ScrollBar.qml 6b6f9dd > src/declarativeimports/plasmacomponents/qml/SectionScroller.qml 9c6cc2e > src/declarativeimports/plasmacomponents/qml/Slider.qml 8441eb5 > src/declarativeimports/plasmacomponents/qml/Switch.qml 60f179d > src/declarativeimports/plasmacomponents/qml/TabBar.qml 809318a > > Diff: https://git.reviewboard.kde.org/r/119163/diff/ > > > Testing > ------- > > http://blogs.fsfe.org/gladhorn/2014/07/07/plasma-next-accessibility/ > > > Thanks, > > Frederik Gladhorn > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
