Bug 83010 - UI: Spinboxes (WrapFields) reset back to 0 after crossing max value
Summary: UI: Spinboxes (WrapFields) reset back to 0 after crossing max value
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
4.4.0.0.alpha0+ Master
Hardware: Other All
: medium minor
Assignee: Caolán McNamara
URL:
Whiteboard: target:4.4.0
Keywords: regression
Depends on:
Blocks:
 
Reported: 2014-08-24 11:16 UTC by Katarina Behrens
Modified: 2016-04-24 16:47 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Katarina Behrens 2014-08-24 11:16:42 UTC
This happens with ca. 1 week old development build, but it's been happening since .ui migration of this tab page, I'd say.

1. Format cells -> Alignment -> Text orientation 

2. Spin 'Degrees' spinbox down to 0 (by clicking on down arrow)

3. Spin down once again, to 359 degrees

4. Try to spin down once again. The value gets reset to 0 (instead of going fluently the full circle to 358, 357, ...)
Comment 1 Katarina Behrens 2014-08-24 11:23:50 UTC
I'm taking this
Comment 2 Katarina Behrens 2014-08-24 16:23:46 UTC
Fun, it's not just cell formatting dialog in Calc, but pretty much all spin boxes that set value of rotation angle (e.g. on multiple places in chart2 - data label formatting, title rotation,...) are affected.
Comment 3 Commit Notification 2014-09-25 16:23:00 UTC
Caolan McNamara committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=e1a5ae3975dfa42003772f8afd98dde387d82653

Related: fdo#83010 move NumericFormatter clipping to min/max into one place



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 4 Commit Notification 2014-09-25 16:23:14 UTC
Caolan McNamara committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=d61fc0b8946eba921d6b9c6d5fcd76fca83c85d7

Related: fdo#83010 implement wrapping on reaching NumericFormatter limits



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 5 Commit Notification 2014-09-25 16:23:28 UTC
Caolan McNamara committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=892a7be963f5ffdb5af86a9de1cf8a613fbdc162

Resolves: fdo#83010 remove svx::WrapField



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 6 Commit Notification 2014-10-01 07:03:56 UTC
Katarina Behrens committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=eb4811590c85895ce531674596bdd6afb3397725

Fallout fdo#83010: wrap on limits also in corner cases



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 7 Jean-Baptiste Faure 2016-04-24 16:47:28 UTC
Version set from description.

Best regards. JBF