[Libreoffice-ux-advise] [Bug 45789] automatic row height in reports
https://bugs.documentfoundation.org/show_bug.cgi?id=45789 Heiko Tietze changed: What|Removed |Added CC|libreoffice-ux-advise@lists |tietze.he...@gmail.com |.freedesktop.org| --- Comment #58 from Heiko Tietze --- No need for input from UX I guess. Removing the ML from CC. -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 45789] automatic row height in reports
https://bugs.documentfoundation.org/show_bug.cgi?id=45789 --- Comment #57 from Howard Johnson --- (In reply to Lionel Elie Mamane from comment #56) > ..the right place to discuss that is the development > mailing list at https://lists.freedesktop.org/mailman/listinfo/libreoffice Thanks. I said >>...report writer, .. extremely slow To briefly follow up on this, after days of looking, I just discovered why one of my base reports was so extremely slow saving and reporting, and how to easily fix it. I'll submit in another bug report. -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 45789] automatic row height in reports
https://bugs.documentfoundation.org/show_bug.cgi?id=45789 --- Comment #56 from Lionel Elie Mamane --- (In reply to Howard Johnson from comment #54) > No really, the more important bug I think, is that the LO source code is > mostly too hard to work on. > MY QUESTION: Are code comment-only commits (i.e. comments w/o any code > changes) welcome to the powers that be here? I expect so. I expect the right place to discuss that is the development mailing list at https://lists.freedesktop.org/mailman/listinfo/libreoffice > 2) As for THIS bug, and having used database report writers (plural) > with CanGrow and CanShrink type features, I am definitely of the opinion > that BOTH of these properties are required, not just AutoGrow, and need > to be added to all controls. Yes, the way I was suggesting this to be implemented is actually equivalent to setting both "CanShrink" and "CanGrow". Setting it per-row can be done relatively easily with the current infrastructure. Setting it per-control requires a deeper redesign of report builder, or a new feature in ODT/Writer. Report builder now "just" creates an ODT document, and Writer does all the actual layouting. So per-control would require that *Writer* support the feature of "autogrow" (which it calls "optimal row height") per cell, and not only per row. -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 45789] automatic row height in reports
https://bugs.documentfoundation.org/show_bug.cgi?id=45789 --- Comment #55 from Ulf Zibis --- (In reply to Howard Johnson from comment #54) > MY QUESTION: Are code comment-only commits (i.e. comments w/o any code > changes) welcome to the powers that be here? > > I realize that the people who wrote this are probably long gone, so any > comments will only try to recapture some of what was in their heads, and > comments will definitely be a work in progress, so some tolerance for not > getting them right the first time would be appreciated. > > Also my call for comments is not just about this bug. It appears to me that > there are a whole backlog of Base bugs that are in the category of: Given > the volunteer time we have available, 'too hard' to fix right now. > > > Can I get a green light to start to add comments to the code to make it more > readable, AND if yes, is there some community tolerance for how this is to > be done, as any comments might be better than none? > > I have some particular commenting styles I've developed over the years, but > even I don't always stick to my commenting styles. :-) For one I never > like processing comments into documentation, so I don't do that. Comments I > think are to help engineers read the code, period. Hi Howard, I appreciate your offer. I'm not one of the deciders here, I just like to comment, that it might be better to open a new bug fot this. You might have more readers then. -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 45789] automatic row height in reports
https://bugs.documentfoundation.org/show_bug.cgi?id=45789 --- Comment #54 from Howard Johnson --- Created attachment 140395 --> https://bugs.documentfoundation.org/attachment.cgi?id=140395&action=edit another, larger database to demo this bug 6 years, 16 volunteers, 4 duplicates, tough bug! (-: My attachment here today is a more comprehensive Base report to further demo this issue. :-) 1) I think we might be starting with the wrong bug. No really, the more important bug I think, is that the LO source code is mostly too hard to work on. I have myself begun to study the source base, but in most places it lacks the most basic comments to help the unfamiliar figure out how it works, to be able to read the code to even work on bugs like this. (A few years go by and I can't even read my own code without a few comments to refresh my memory. And as we all get older, this issue gets worse.) So what I think is needed _first_, is to work together (or alone) to start to add sufficient comments to the code base to help new coders find their way around. One needs to be able to read the code without going to an extreme amount of work. I am willing to help with such an effort, as I begin to try and read the code myself. (Please see more specific question about this below.) 2) As for THIS bug, and having used database report writers (plural) with CanGrow and CanShrink type features, I am definitely of the opinion that BOTH of these properties are required, not just AutoGrow, and need to be added to all controls. The reason for CanShrink is that if no data is in group, it is often desirable to fully collapse the group, so as to save paper space where it's not needed. But this can only reasonably work if CanShrink is also available. In this case you would set the can-shrink on all fields in the group, and also on the group itself. Furthermore, I think it would make no sense to fix only half of this problem, and to miss the other half, especially when it is so hard to do, and these are so closely related. 3) Finally, and sadly, the Base report writer, as it is, is extremely slow. Just try to produce a report with 100 pages sometime. It takes forever. So I think any work done on #1 or #2 above should keep a watchful eye on someday re-writing the slower parts, (or the whole thing) to be faster. For one, I can see from the above comments that Java is involved. At a minimum I think that it someday needs to get rewritten in some flavor of C, as I have found java to sometimes be slow. SUMMARY: So my point #1 above (call for code comments), becomes even more important in the long haul: It will help fix things now, and make things faster in the future. MY QUESTION: Are code comment-only commits (i.e. comments w/o any code changes) welcome to the powers that be here? I realize that the people who wrote this are probably long gone, so any comments will only try to recapture some of what was in their heads, and comments will definitely be a work in progress, so some tolerance for not getting them right the first time would be appreciated. Also my call for comments is not just about this bug. It appears to me that there are a whole backlog of Base bugs that are in the category of: Given the volunteer time we have available, 'too hard' to fix right now. Can I get a green light to start to add comments to the code to make it more readable, AND if yes, is there some community tolerance for how this is to be done, as any comments might be better than none? I have some particular commenting styles I've developed over the years, but even I don't always stick to my commenting styles. :-) For one I never like processing comments into documentation, so I don't do that. Comments I think are to help engineers read the code, period. Thanks. I'm so independent that I often won't do what I put on my own to-do list. :-~ -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 45789] automatic row height in reports
https://bugs.documentfoundation.org/show_bug.cgi?id=45789 Howard Johnson changed: What|Removed |Added CC||BridgeportContractor@gmail. ||com --- Comment #53 from Howard Johnson --- *** Bug 116210 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 45789] automatic row height in reports
https://bugs.documentfoundation.org/show_bug.cgi?id=45789 --- Comment #52 from gdaheffer...@hotmail.com --- Thank you Alex for spotting and e-mailing me that my enhancement Bug 114387 has already been submitted as 45789. I apologize for any inconvenience caused. -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 45789] automatic row height in reports
https://bugs.documentfoundation.org/show_bug.cgi?id=45789 Alex Thurgood changed: What|Removed |Added CC||gdaheffer...@hotmail.com --- Comment #51 from Alex Thurgood --- *** Bug 114387 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 45789] automatic row height in reports
https://bugs.documentfoundation.org/show_bug.cgi?id=45789 rob...@familiegrosskopf.de changed: What|Removed |Added See Also||https://bugs.documentfounda ||tion.org/show_bug.cgi?id=11 ||4387 -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 45789] automatic row height in reports
https://bugs.documentfoundation.org/show_bug.cgi?id=45789 jan iversen changed: What|Removed |Added CC||libreoffice-ux-advise@lists ||.freedesktop.org -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise