Created attachment 144451 [details] Contextmenu Draw for Writer, Calc, Draw and Impress Contextual menu when you draw something and use right click in the different LibO apps: writer ------ - Insert Caption to the bottom like in graphic - Group submenu is always empty and should not be available (only LeaveGroup command) - Flip change to Rotate or Flip like in graphic where you have also the rotate shape feature calc ---- - Layout similar to what graphic contextmenu show (BUG:119412) and what draw contextmenu from writer look like draw ---- - Add Points cause they are available in writer - Flip added also the rotate features so it's Rotate or Flip - Transformations was added cause this features for similar to rotate but more advanced and draw is the "shape" app - Text Box like in writer impress ------- - look like draw + Interaction
Would add Base ------- Embedded Form: The context menu is same as a stand alone Writer file and includes 'add caption'item which IMO has no meaning here. Report: The context here has none of the items you would expect for a shape.
(In reply to andreas_k from comment #0) > writer > ------ > - Group submenu is always empty and should not be available (only LeaveGroup > command) It isn't empty if you select 2 shapes. > draw > ---- > - Text Box like in writer I'm not against adding this entry in Draw, just want to point out that it won't have the same effect as in Writer. The "Add Text Box" command in Writer is actually about attaching a Writer text frame to the given shape, not about activating the shape built-in text engine like in Draw. It's not the same as if you double click the shape, or press F2. That's why I think we should rename this "Add Text Box" to "Add Text Frame" to avoid further confusion. Also a general note about Impress/Draw: Unlike Writer/Calc which have a single "draw" context menu for all kinds of shapes, Impress/Draw have separate context menus for lines, curves, measure lines etc. So whenever we change the context menu structure, it should be reflected in all those popupmenus as well, for consistency. On the other hand, if you could propose a single "draw" menu structure that could work across several kinds of shapes, I'll be happy to look at making the necessary code changes to use such a unified menu. Just let me know.
> > draw > > ---- > > - Text Box like in writer > I'm not against adding this entry in Draw, just want to point out that it > won't have the same effect as in Writer. The "Add Text Box" command in > Writer is actually about attaching a Writer text frame to the given shape, > not about activating the shape built-in text engine like in Draw. It's not > the same as if you double click the shape, or press F2. That's why I think > we should rename this "Add Text Box" to "Add Text Frame" to avoid further > confusion. I would keep it there "only" for consistency in addition it's alway nice to see in the UI that you can add text to whatever shape use draw. > Also a general note about Impress/Draw: > > Unlike Writer/Calc which have a single "draw" context menu for all kinds of > shapes, Impress/Draw have separate context menus for lines, curves, measure > lines etc. So whenever we change the context menu structure, it should be > reflected in all those popupmenus as well, for consistency. On the other > hand, if you could propose a single "draw" menu structure that could work > across several kinds of shapes, I'll be happy to look at making the > necessary code changes to use such a unified menu. Just let me know. I will investigate to reduce the different .xml popupmenu files for different shapes cause it's way easier in writer and I also think (like you) this complexity (maintanance work) can be reduced.
Created attachment 144468 [details] Contextmenu Draw for Writer, Calc, Draw and Impress v2
(In reply to andreas_k from comment #4) > Created attachment 144468 [details] > Contextmenu Draw for Writer, Calc, Draw and Impress v2 Any chance you could let me know which source XML files you are working with, perhaps I can work on the Base versions. Thanks
> Any chance you could let me know which source XML files you are working > with, perhaps I can work on the Base versions. I'm not very familar with base so your help is very welcome. I think the context menu files for the reporter are here. core/sw/uiconfig/swreport/popupmenu/
(In reply to andreas_k from comment #6) > > Any chance you could let me know which source XML files you are working > > with, perhaps I can work on the Base versions. > > I'm not very familar with base so your help is very welcome. I think the > context menu files for the reporter are here. > > core/sw/uiconfig/swreport/popupmenu/ I'll see what I can make of that. Meantime started an image only mockup of what I thought made sense yesterday. Could attach that here, with some explanation, later today, if you like. I have not built LibrOffice in many years but the files are coming down my machine as I type this, so will get off to looking at the files when that is done. Thanks much.
For Writer: Align - doesn't work, cause you cannot select more one shape
(In reply to Roman Kuznetsov from comment #8) > For Writer: > > Align - doesn't work, cause you cannot select more one shape You can use AlignFrameMenu in the image the wrong align element was shown.
And for select more than one graphic I suggest Bug 119526
Created attachment 144529 [details] Contextmenu Draw for Writer, Calc, Draw and Impress v3 Same layout than all the other context menues. First Points cause this it the action where you can edit shapes. Than Position, line and area where you define the layout and where it was located. Than all the align, arrange, rotate, transforms and convert items (in writer and calc only rotate and flip is available in draw and impress all is available) Last Description and Style
I suggest we get rid of “Rotate or Flip…” - it is simply not OK (for the same reason we are not using “Copy, Cut or Paste…” or “Arrange or Align…” as a command with submenu in context menus). If no other better solution pops up, I suggest to rename this submenu title with "Transform...".
Seems to me that this effort is appreciated though details are still in discussion.
(In reply to Martin Srebotnjak from comment #12) > I suggest we get rid of “Rotate or Flip…” - it is simply not OK Thanks for the feedback I already have an separate bug report for this toppic BUG 119410
(In reply to Martin Srebotnjak from comment #12) > I suggest we get rid of “Rotate or Flip…” - it is simply not OK (for the > same reason we are not using “Copy, Cut or Paste…” or “Arrange or Align…” as > a command with submenu in context menus). > If no other better solution pops up, I suggest to rename this submenu title > with "Transform...". Transformations is still available in draw and impress. Where rotate is also available but not "simple" mirror. Rotate and Flip is already available in writer for graphic context menu.
(In reply to Martin Srebotnjak from comment #12) > I suggest we get rid of “Rotate or Flip…” - it is simply not OK (for the > same reason we are not using “Copy, Cut or Paste…” or “Arrange or Align…” as > a command with submenu in context menus). > If no other better solution pops up, I suggest to rename this submenu title > with "Transform...". As there is a Transformations sub-menu in Draw's draw context menu (see https://colibreoffice.files.wordpress.com/2018/08/contextmenudraw.png?w=1100), Transform is not usable
Use Flip as it is now. So no Change from Flip to Rotate or Flip. If the suggestion is to combine it in BUG 119410 I will add an additional patch.
andreas kainz committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=cf8b3a74e8c093dd0ffc7e2f431a26864958f6fb tdf#119506 Draw Contextmenu in Writer, Calc, Draw, Impress It will be available in 6.2.0. 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.
Thanks, Andreas! Verified fixed. Version: 6.2.0.0.alpha1+ Build ID: 5d2ab49cbda1d7aea1019478abe0163e1f40a121 CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; Locale: de-DE (de_DE.UTF-8); Calc: threaded from today