Bug 34517 - LibO 3.3.1RC2 UI: Texts in context menu 'Arrangement for chart lines' interchanged
Summary: LibO 3.3.1RC2 UI: Texts in context menu 'Arrangement for chart lines' interch...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Chart (show other bugs)
Version: unspecified
Hardware: All All
: medium normal
Assignee: André Schnabel
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-20 23:34 UTC by Rainer Bielefeld Retired
Modified: 2013-05-03 21:41 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
See original report! (210.00 KB, application/x-download)
2011-02-20 23:34 UTC, Rainer Bielefeld Retired
Details

Description Rainer Bielefeld Retired 2011-02-20 23:34:36 UTC
Created attachment 43587 [details]
See original report!

I found this with "LibreOffice 3.3.1 RC2 - WIN7  Home Premium (64bit) English UI  [OOO330m19 (build 7 / tag 3.3.1.1)]" during my tests for 
Bug 34431 - More symbols (crosses, circles) needed for data points in charts

Steps to reproduce:
1. open sample document "freestylesymbols" "from Bug 34431
  you should see line with coffee beans behind line with envelops
2. to get coffee beans in front:
   double click chart
3. Click a coffee bean, so that line becomes selectd (control points appear)
4. rightclick to get context menu, > Arrangement
   Expected: because line is backward, there should be "Forward"
   Actual: Text "Send backward"
5. Proceed:
   Expected: only possible action "Send Forward" should be done
   Actual: as expected

So the texts of the context menu "Arrangement" are interchanged 
  
I see the same problem with " Ooo 3.3.0  – WIN7  Home Premium  (64bit) German UI [OOO330m20 (build 9567)]"
Comment 1 André Schnabel 2011-08-02 09:58:19 UTC
confirmed using LibO 3.4.2 on Windows and Linux

quickly checking the string ID's in pootle, it seems that the implementation is wrong. I see no "backward" string used for an "forward" ID.

after some more analysis it seems, as if the order of series is wrong in
 calc/chart2/sourcetools/DiagramHelper.cxx lcl_moveSeriesOrCheckIfMoveIsAllowed
lines 1359 to 1361
Comment 2 André Schnabel 2011-08-02 10:52:23 UTC
taking this one, as the suggested fix really seems to solve the problem but I need to do some more testing
Comment 3 Björn Michaelsen 2011-12-23 13:26:24 UTC
Since all new unconfirmed bugs start in state UNCONFIRMED now and old unconfirmed bugs were moved to NEEDINFO with a explanatory comment, all bugs promoted above those bug states to NEW and later are automatically confirmed making the CONFIRMED whiteboard status redundant. Thus it will be removed.
Comment 4 James 2013-05-03 21:41:20 UTC
Tested this. Still valid for LO 4.0.3.1 RC.

Removing André from "Assigned to:" since this has been open for over a year. Setting to new.

Feel free to re-take if you still want to work on this André.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.