Description: Add more tools to the new Table section in the Writer Sidebar After https://gerrit.libreoffice.org/#/c/81893/ we have the nice Table section in Sidebar. But it has a few tools for me. May be it needs add more tools to the section? This enhancement just for discuss with UX - team. I suggest to add next tools: More options button to heading of the Table section (as is for Paragraph and Character sections) Split Cells Select Table Table Cell Background Color Borders Formula (F2) Steps to Reproduce: - Actual Results: - Expected Results: - Reproducible: Always User Profile Reset: No Additional Info: -
Keep in mind that we have a toolbar to start functions, the sidebar should not be used as a replacement for the properties dialog. (In reply to Roman Kuznetsov from comment #0) > Split Cells We have Merge and I would rather remove this button. > Select Table Would run this from the toolbar. > Table Cell Background Color > Borders > Formula (F2) Toolbar as well. And "Delete Table" (as well as Delete Col/Row) is also not well suited at the sidebar. Keep it simple.
Would appreciate more comments here.
Tomaz, what's your opinion?
It was Tamas ;-)
Created attachment 155948 [details] Screen
(In reply to Heiko Tietze from comment #1) > Keep in mind that we have a toolbar to start functions, the sidebar should > not be used as a replacement for the properties dialog. > > (In reply to Roman Kuznetsov from comment #0) > > Split Cells > We have Merge and I would rather remove this button. > > > Select Table > Would run this from the toolbar. > > > Table Cell Background Color > > Borders > > Formula (F2) > Toolbar as well. > > And "Delete Table" (as well as Delete Col/Row) is also not well suited at > the sidebar. Keep it simple. The buttons "select cell" and "select column", like the rest, should be in the sidebar when editing the table. That would not turn out a triangle cursor path to the entire monitor, as in the picture. The text was written through a translator.
Mouse travel is a good argument (I also just wanted to mention simplicity). So let's add Select, Split, and Merge.
Created attachment 156202 [details] Mockup. Left - before, right - after adding
Added my variant of tools on Table section
Roman Kuznetsov committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/d068c68640f32d4a94da4df197441ee09a702b75 tdf#128683 Add more tools to Table section in Writer's Sidebar It will be available in 6.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
>More options button to heading of the Table section (as is for Paragraph and Character sections) - still missing! >Split Cells - done >Select Table - done >Table Cell Background Color - done >Borders - done >Formula (F2) - done this enh still want a love from a real developer!
(In reply to Roman Kuznetsov from comment #11) > >More options button to heading of the Table section (as is for Paragraph and Character sections) - still missing! > > >Split Cells - done > >Select Table - done > >Table Cell Background Color - done > >Borders - done > >Formula (F2) - done > > this enh still want a love from a real developer! Do these items have a working implementation? I don't like the idea that you are adding GUI items to the sidebar without they really work. For example, I needed to extend the C++ code to have the right initial state for the sidebar buttons on this panel: https://cgit.freedesktop.org/libreoffice/core/commit/?id=47c88b2c4db23f33a4371139745b8427564d3667
(In reply to Tamás Zolnai from comment #12) > (In reply to Roman Kuznetsov from comment #11) > > >More options button to heading of the Table section (as is for Paragraph and Character sections) - still missing! > > > > >Split Cells - done > > >Select Table - done > > >Table Cell Background Color - done > > >Borders - done > > >Formula (F2) - done > > > > this enh still want a love from a real developer! > > Do these items have a working implementation? I don't like the idea that you > are adding GUI items to the sidebar without they really work. it worked when I checked it on my build. Of course I don't want add UI items that don't work. > For example, I needed to extend the C++ code to have the right initial state > for the sidebar buttons on this panel: > https://cgit.freedesktop.org/libreoffice/core/commit/ > ?id=47c88b2c4db23f33a4371139745b8427564d3667 If I made something wrong then I'll fix it of course. If you have any other notes for my patch, then please write it. Thank you for feedback ps: I should was add you to reviewer list in the patch =(
Tamás, it works for me. I don't see what's wrong there
Roman Kuznetsov committed a patch related to this issue. It has been pushed to "libreoffice-6-4": https://git.libreoffice.org/core/commit/aa86a2e30de9e9433dcc41c09023a07945affaf9 tdf#128683 Add more tools to Table section in Writer's Sidebar It will be available in 6.4.0.1. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
(In reply to Roman Kuznetsov from comment #14) > Tamás, it works for me. I don't see what's wrong there Ok, I checked it. One issue I found easily is the state of the Split Table button. When you click in the first row of a table the Split Table button is disabled on the toolbar and should be disabled also on the sidebar. The problem comes up when you first click outside of the table (so the Table sidebar panel disappear) and then click inside the table, in the first row. The Split Table button is disabled on the toolbar, but not disabled on the sidebar panel. So this initial state, when you first click into a table, is the problem here. I'm not sure whether a similar issue might happen to other newly added buttons or not. Hard to tell whether there is any condition when they are should be disabled.
(In reply to Tamás Zolnai from comment #16) > Ok, I checked it. > One issue I found easily is the state of the Split Table button. When you > click in the first row of a table the Split Table button is disabled on the > toolbar and should be disabled also on the sidebar. > The problem comes up when you first click outside of the table (so the Table > sidebar panel disappear) and then click inside the table, in the first row. > The Split Table button is disabled on the toolbar, but not disabled on the > sidebar panel. > So this initial state, when you first click into a table, is the problem > here. I'm not sure whether a similar issue might happen to other newly added > buttons or not. Hard to tell whether there is any condition when they are > should be disabled. Now I see that problem. But I need some time for investigations..
Roman Kuznetsov committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/82dce922e6d36b40dd0571b0c00666cd6fc1a8ac tdf#128683 Add More Options button to Table section in Writer's Sidebar It will be available in 6.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Roman Kuznetsov committed a patch related to this issue. It has been pushed to "libreoffice-6-4": https://git.libreoffice.org/core/commit/868f3b157e2982887c3efeb4de850560a43755c5 tdf#128683 Add More Options button to Table section in Writer's Sidebar It will be available in 6.4.0.2. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Verified in Version: 6.5.0.0.alpha0+ (x64) Build ID: 350d25da375f221edfa37309324ce3c68cf297ef CPU threads: 4; OS: Windows 10.0 Build 18362; UI render: default; VCL: win; Locale: de-DE (de_DE); UI-Language: en-GB Calc: threaded Although not all tools mentioned in the initial enhancement request are implemented, I change the bug to VERFIIED, because I understand the status RESOLVED FIXED in that way, that Roman finished his work. Works fine. Roman, tanks for those improvements.