Steps to reproduce:
1. Import Visio document with multiple sheets (I had 50).
2. View import in LO Draw.
All sheets are imported, but not in the same order as they are in Visio. Also, in text fields from Visio that had black fonts, they became yellow in Draw.
Expected behavior: Keep the same order of sheets, as well as the same color of fonts.
Platform (if different from the browser):
x86 32-bit, running Win XP Pro. LO version 126.96.36.199.
Created attachment 62058 [details]
Attachment is a fairly simple Visio document, only 5 sheets, but with enough complexity to reproduce both problems.
1. Save Visio drawing on your computer.
2. Open LO Draw.
3. File >> Open.
4. Browse to Visio drawing.
5. Select Visio drawing.
6. Click Open.
Expected: sheets in same order as in Visio (1,2,3,4,5). Fonts same color as in Visio. But, what you see instead are sheets in a different order (2,4,5,3,1), as well as a number of font color changes: the sheet coordinates are changed from black to yellow, some of the info in the lower-right-hand corner is changed from black to green, and some of the text in the legend box in the lower-left-hand corner is changed from black to blue.
Page order problem is clear.
Text colour needs some additional investigation -- LibreOffice does use colours that are actually stored in attached VSD file.
@Valek: Thanks for reply.
You write, "Page order problem is clear." Do you mean the problem statement is clear, or that the problem solution is clear?
You write, "Text colour needs some additional investigation..." Should I do some investigation? I haven't dived into LO development at all. I could try to narrow it down a bit - a more minimal example that still reproduces the problem, if that would help.
Solution is clear for page order.
For text clr I will check why it was overridden by black, no action is required from you at the moment.
Of course if you want to contribute to LibreOffice, you are always welcome!
Check http://www.libreoffice.org/developers-2/ for ways to contribute.
Colours for text/fill/stroke can be set (or locked to defaults) by layer.
libvisio needs to parse LayerIX chunk (probably only layers with colour/transparency and "visible" flag are important at the moment) and LayerMem chunk to apply layer props as required.
Ok, sounds good! Thanks for the clarifications.
Not ready for development work as yet, but I'll keep that in mind. I'm relearning my C right now.
OK, I fixed the page order in libvisio git and will be in the next release, which means that this fix will be for sure in LibreOffice 3.6.0. The text colour stuff is a bit more complicated and I will try to have a run on it too, just not sure how much time it will take.
As an update: this 50-sheet document I had imported earlier, all of a sudden shuffled its sheets into some new order, not the one I had put them in (corresponding to the Visio document). I'm wondering if your bug fix will fix this as well.
Then you create a document in Visio and start to add pages to it, Visio increment page IDs for each new page.
If you remove some pages from the start or shuffle pages, Visio do not re-numerate them internally. To keep proper (for representation) order Visio stores ordered list of page IDs.
Before Fridrich's fix libvisio was not taking this list into account but read pages in the sequence it's stored in the file (with weird results you observed).
With this fix libvisio uses the list to throw pages in the right order.
Thanks for the explanation. So is there a way in LO Draw 3.5 to number the sheets so they stay put in the order I want them in? I suppose one way would be to start a brand-new document in Draw, and then copy and paste the objects into the new file. Would that work, or would these page ID's still come along for the ride?
Please disregard my last comment. I found out today that I was accidentally re-opening the Visio document, instead of the LO Draw document; I was just seeing the bug exactly as originally reported. My LO Draw document has not reshuffled its pages.
Sorry about the confusion!
Any update with new LO version (4.0.5 or 4.1.0)? I opened the document on pc Debian x86-64 with master sources updated yesterday but didn't see (certainly missed) the point.
(In reply to comment #12)
> Any update with new LO version (4.0.5 or 4.1.0)? I opened the document on pc
> Debian x86-64 with master sources updated yesterday but didn't see
> (certainly missed) the point.
Page order was fixed while ago.
Layers are not supported yet, hence color part is not fixed.
** Please read this message in its entirety before responding **
To make sure we're focusing on the bugs that affect our users today, LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed bugs which have not been touched for over a year.
There have been thousands of bug fixes and commits since anyone checked on this bug report. During that time, it's possible that the bug has been fixed, or the details of the problem have changed. We'd really appreciate your help in getting confirmation that the bug is still present.
If you have time, please do the following:
*Test to see if the bug is still present on a currently supported version of LibreOffice (4.4.1 or later)
*If the bug is present, please leave a comment that includes the version of LibreOffice and your operating system, and any changes you see in the bug behavior
*If the bug is NOT present, please set the bug's Status field to RESOLVED-WORKSFORME and leave a short comment that includes your version of LibreOffice and Operating System
Please DO NOT
*Update the version field
*Reply via email (please reply directly on the bug tracker)
*Set the bug's Status field to RESOLVED - FIXED (this status has a particular meaning that is not appropriate in this case)
If you want to do more to help you can test to see if your issue is a REGRESSION. To do so:
1. Download and install oldest version of LibreOffice (usually 3.3 unless your bug pertains to a feature added after 3.3)
2. Test your bug
3. Leave a comment with your results.
4a. If the bug was present with 3.3 - set version to "inherited from OOo";
4b. If the bug was not present in 3.3 - add "regression" to keyword
Feel free to come ask questions or to say hello in our QA chat: http://webchat.freenode.net/?channels=libreoffice-qa
Thank you for your help!
-- The LibreOffice QA Team This NEW Message was generated on: 2015-04-18
(In reply to Valek Filippov from comment #13)
> (In reply to comment #12)
> > Any update with new LO version (4.0.5 or 4.1.0)? I opened the document on pc
> > Debian x86-64 with master sources updated yesterday but didn't see
> > (certainly missed) the point.
> Page order was fixed while ago.
> Layers are not supported yet, hence color part is not fixed.
Yep, colors still wrong.
Win 7 Pro 64-bit Version: 188.8.131.52.alpha1+
Build ID: 01a189abcd9a4ca472a74b3b2c000c9338fc2c91
TinderBox: Win-x86@39, Branch:master, Time: 2015-06-14_07:46:28
Locale: fi-FI (fi_FI)
With the commit
libvisio now considers the layer colour information and this bug is fixed. We need a libvisio release and integration in LO for it to bubble up into LO. But I consider this one fixed now.