Download it now!
Bug 136073 - Sidebar: effect section instead of glow, shadow, soft edge separate section
Summary: Sidebar: effect section instead of glow, shadow, soft edge separate section
Status: ASSIGNED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: All All
: medium enhancement
Assignee: andreas_k
URL:
Whiteboard: target:7.1.0
Keywords:
Depends on:
Blocks: Sidebar-UI-UX
  Show dependency treegraph
 
Reported: 2020-08-24 13:58 UTC by andreas_k
Modified: 2020-10-23 06:46 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
effect section (37.83 KB, image/png)
2020-08-24 13:58 UTC, andreas_k
Details
8 sections in impress (18.39 KB, image/png)
2020-08-24 13:58 UTC, andreas_k
Details
chart Type sidebar layout (17.01 KB, image/png)
2020-08-28 20:25 UTC, andreas_k
Details

Note You need to log in before you can comment on or make changes to this bug.
Description andreas_k 2020-08-24 13:58:09 UTC
Created attachment 164634 [details]
effect section

glow, shadow and soft edge are separate sidebar sections. glow and soft edge are new in 7.0.

after a discussion with Mike the suggestion is to group all 3 sections into one effect section. 

This has the following benefit:
- 2 sections less (in impress there are now 8 sections)
- better alignment cause the widgets can be aligned with the string "Transparancy:"
- reduce of complexity

at the attachment you see the proposal. I'm open with the sorting of the 3 sections. From the code point of view it work opposit way.
Comment 1 andreas_k 2020-08-24 13:58:39 UTC
Created attachment 164635 [details]
8 sections in impress
Comment 2 andreas_k 2020-08-28 20:25:44 UTC
Created attachment 164827 [details]
chart Type sidebar layout

The Chart Sidebar Elements group can be used for a layout proposal for effect sidebar group.
Comment 3 Heiko Tietze 2020-09-08 08:04:14 UTC
Disclaimer in bug 136125 comment 3 - remove all effects from the sidebar but keep in Draw/Impress. And a common section for all effects features make sense, although it's pointless to set a blurry shadow together with glow and/or soft edges. 

So again: Remove this in Writer and combine in Draw/Impress.
Comment 4 andreas_k 2020-09-08 08:17:09 UTC
how the effect section can look like you will see in attachment 164802 [details]
Comment 5 Mike Kaganski 2020-09-08 08:19:07 UTC
(In reply to Heiko Tietze from comment #3)
> although it's pointless to set a blurry shadow together with glow
> and/or soft edges. 

FTR: It is not pointless. Shadow of a semi-transparent soft-edged object with semi-transparent glow gets additional effects in its shadow by applying blur.
Comment 6 andreas_k 2020-09-08 08:23:08 UTC
FTR: If the effect stuff is available in writer, I don't have a problem to show an CLOSED (by default) effect section in the sidebar. It's consistent and show that LibO is ONE product and you can use stuff in writer, calc, impress, draw, ...

but please merge the 3 sections into one.
Comment 7 Roman Kuznetsov 2020-09-08 08:29:06 UTC
(In reply to Heiko Tietze from comment #3)
> Disclaimer in bug 136125 comment 3 - remove all effects from the sidebar but
> keep in Draw/Impress. And a common section for all effects features make
> sense, although it's pointless to set a blurry shadow together with glow
> and/or soft edges. 
> 
> So again: Remove this in Writer and combine in Draw/Impress.

And if I'll want add Glow to shape in Writer I'll can't just use Sidebar and should be use a dialog (what's dialog, btw?) that doesn't have these options?!
Comment 8 Heiko Tietze 2020-09-08 08:39:02 UTC
(In reply to Roman Kuznetsov from comment #7)
> And if I'll want add Glow to shape in Writer I'll can't just use Sidebar and
> should be use a dialog (what's dialog, btw?) that doesn't have these
> options?!

Which is a serious bug. Please file a ticket.
Comment 9 Commit Notification 2020-10-19 11:06:50 UTC
andreas kainz committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/55281fe6d3cda23d37b0b0c368786c9fc4c5abe9

tdf#136073 combine glow and softedge sidebar section

It will be available in 7.1.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.
Comment 10 Heiko Tietze 2020-10-20 09:25:47 UTC
Follow-up in bug 137615; please reopen in case the patch is reverted eventually.
Comment 11 andreas_k 2020-10-20 09:32:26 UTC
(In reply to Heiko Tietze from comment #10)
> Follow-up in bug 137615; please reopen in case the patch is reverted
> eventually.

Only Soft Edge and Glow are in the Effect Section Sidebar. Shadow need to be added but need more work, cause Soft Edge and Glow are not everywhere available, when Shadow is available.
Comment 12 Heiko Tietze 2020-10-20 09:38:47 UTC
I guess you will keep on with the integration, once the crash is fixed => ASSIGNED
Comment 13 andreas_k 2020-10-23 06:46:33 UTC
(In reply to Commit Notification from comment #9)
> andreas kainz committed a patch related to this issue.
> It has been pushed to "master":
> 
> https://git.libreoffice.org/core/commit/
> 55281fe6d3cda23d37b0b0c368786c9fc4c5abe9
> 
> tdf#136073 combine glow and softedge sidebar section
> 
> It will be available in 7.1.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.

After I submit the patch to combine Glow and Soft Edge Sidebar Section, when I will open the new effect sidebar section, LibO crash.