Re: Review Request 110176: Change the visual appearance of the composited outline
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/#review31832 --- kwin/outline.cpp http://git.reviewboard.kde.org/r/110176/#comment23735 though not sure about this - personally liked the other variant better (also, what if there's a transparent panel on either edge?) - Thomas Lübking On April 26, 2013, 6 a.m., Martin Gräßlin wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/ --- (Updated April 26, 2013, 6 a.m.) Review request for kwin and Plasma. Description --- Change the visual appearance of the composited outline Uses widgets/translucentbackground as FrameSvg item to ensure that we don't get a huge black square on the screen. When bordering a screen edge we disable the border except if all edges are bordered. This makes a little bit more clear in the quick tiling case what will be the geometry. Blur behind the outline does not work as: * moving the window generates artefacts * moved window needs to be above the outline, but outline needs to be on top of everything. Elevating the moved window results in strange side effects when moving the window below a keep above, etc. Diffs - kwin/geometry.cpp 5a6786d kwin/outline.cpp ad3cfc0 Diff: http://git.reviewboard.kde.org/r/110176/diff/ Testing --- I tried various Plasma theme elements. dialog/* and widgets/background are not useable as there are solid themes. Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 110176: Change the visual appearance of the composited outline
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/#review31844 --- This review has been submitted with commit b76c40a58bc530b3d05530878e6e74d1105b396c by Martin Gräßlin to branch master. - Commit Hook On April 26, 2013, 6 a.m., Martin Gräßlin wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/ --- (Updated April 26, 2013, 6 a.m.) Review request for kwin and Plasma. Description --- Change the visual appearance of the composited outline Uses widgets/translucentbackground as FrameSvg item to ensure that we don't get a huge black square on the screen. When bordering a screen edge we disable the border except if all edges are bordered. This makes a little bit more clear in the quick tiling case what will be the geometry. Blur behind the outline does not work as: * moving the window generates artefacts * moved window needs to be above the outline, but outline needs to be on top of everything. Elevating the moved window results in strange side effects when moving the window below a keep above, etc. Diffs - kwin/geometry.cpp 5a6786d kwin/outline.cpp ad3cfc0 Diff: http://git.reviewboard.kde.org/r/110176/diff/ Testing --- I tried various Plasma theme elements. dialog/* and widgets/background are not useable as there are solid themes. Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 110176: Change the visual appearance of the composited outline
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/#review31812 --- Ship it! elevating the dragged window makes it acceptable even with opaque themes and a clear improvement over the status quo in about all cases. - Thomas Lübking On April 26, 2013, 6 a.m., Martin Gräßlin wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/ --- (Updated April 26, 2013, 6 a.m.) Review request for kwin and Plasma. Description --- Change the visual appearance of the composited outline Uses widgets/translucentbackground as FrameSvg item to ensure that we don't get a huge black square on the screen. When bordering a screen edge we disable the border except if all edges are bordered. This makes a little bit more clear in the quick tiling case what will be the geometry. Blur behind the outline does not work as: * moving the window generates artefacts * moved window needs to be above the outline, but outline needs to be on top of everything. Elevating the moved window results in strange side effects when moving the window below a keep above, etc. Diffs - kwin/geometry.cpp 5a6786d kwin/outline.cpp ad3cfc0 Diff: http://git.reviewboard.kde.org/r/110176/diff/ Testing --- I tried various Plasma theme elements. dialog/* and widgets/background are not useable as there are solid themes. Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 110176: Change the visual appearance of the composited outline
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/ --- (Updated April 26, 2013, 8 a.m.) Review request for kwin and Plasma. Changes --- elevated the moved window while outline is shown. Please give it a try. Description --- Change the visual appearance of the composited outline Uses widgets/translucentbackground as FrameSvg item to ensure that we don't get a huge black square on the screen. When bordering a screen edge we disable the border except if all edges are bordered. This makes a little bit more clear in the quick tiling case what will be the geometry. Blur behind the outline does not work as: * moving the window generates artefacts * moved window needs to be above the outline, but outline needs to be on top of everything. Elevating the moved window results in strange side effects when moving the window below a keep above, etc. Diffs (updated) - kwin/geometry.cpp 5a6786d kwin/outline.cpp ad3cfc0 Diff: http://git.reviewboard.kde.org/r/110176/diff/ Testing --- I tried various Plasma theme elements. dialog/* and widgets/background are not useable as there are solid themes. Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 110176: Change the visual appearance of the composited outline
On April 25, 2013, 11:10 a.m., Marco Martin wrote: seems a nice idea. since the shadow for that item is huge, does take it into account when it resizes the frame? )you would want the frame border, not the shadow more or less aligned with the window size) no the border is not considered, but I think that's just fine. It's nothing you really notice that it is not exactly the same geometry and after all: it's just an outline. - Martin --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/#review31539 --- On April 26, 2013, 8 a.m., Martin Gräßlin wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/ --- (Updated April 26, 2013, 8 a.m.) Review request for kwin and Plasma. Description --- Change the visual appearance of the composited outline Uses widgets/translucentbackground as FrameSvg item to ensure that we don't get a huge black square on the screen. When bordering a screen edge we disable the border except if all edges are bordered. This makes a little bit more clear in the quick tiling case what will be the geometry. Blur behind the outline does not work as: * moving the window generates artefacts * moved window needs to be above the outline, but outline needs to be on top of everything. Elevating the moved window results in strange side effects when moving the window below a keep above, etc. Diffs - kwin/geometry.cpp 5a6786d kwin/outline.cpp ad3cfc0 Diff: http://git.reviewboard.kde.org/r/110176/diff/ Testing --- I tried various Plasma theme elements. dialog/* and widgets/background are not useable as there are solid themes. Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 110176: Change the visual appearance of the composited outline
On April 25, 2013, 9:10 a.m., Marco Martin wrote: seems a nice idea. since the shadow for that item is huge, does take it into account when it resizes the frame? )you would want the frame border, not the shadow more or less aligned with the window size) Martin Gräßlin wrote: no the border is not considered, but I think that's just fine. It's nothing you really notice that it is not exactly the same geometry and after all: it's just an outline. I'd frankly leave the shadow padding to prevent cutting of rounded edges - and eventually dep. on theme stress this is not a bug ;-) - Thomas --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/#review31539 --- On April 26, 2013, 6 a.m., Martin Gräßlin wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/ --- (Updated April 26, 2013, 6 a.m.) Review request for kwin and Plasma. Description --- Change the visual appearance of the composited outline Uses widgets/translucentbackground as FrameSvg item to ensure that we don't get a huge black square on the screen. When bordering a screen edge we disable the border except if all edges are bordered. This makes a little bit more clear in the quick tiling case what will be the geometry. Blur behind the outline does not work as: * moving the window generates artefacts * moved window needs to be above the outline, but outline needs to be on top of everything. Elevating the moved window results in strange side effects when moving the window below a keep above, etc. Diffs - kwin/geometry.cpp 5a6786d kwin/outline.cpp ad3cfc0 Diff: http://git.reviewboard.kde.org/r/110176/diff/ Testing --- I tried various Plasma theme elements. dialog/* and widgets/background are not useable as there are solid themes. Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 110176: Change the visual appearance of the composited outline
On April 25, 2013, 12:29 p.m., Thomas Lübking wrote: It's a definitively visual improvement with many themes (including and esp. air), but eg. (old) oxygen and several others still get me a black overlay that does not much look like a frame or window and with opaquity it's (surprise) *opaque* (ie. one side of the screen turns -near- black) http://techbase.kde.org/Development/Tutorials/Plasma/ThemeDetails /translucentbackground.svg: a standard background image for plasmoids that for their nature are bigger and with not much text. In this case a translucent background looks better. It needs the same elements of background.svg in it. If this file is not present, the plasmoids that uses this will use background.svg instead. There's (unfortunately) no explicit requirement on that either this is translucent or exists at all (what will get us the regular solid background) For the beginning, i'd strongly recommend to elevate the to-be-tiled window over the outline. Martin Gräßlin wrote: @Plasma devs: could you please clarify whether there is a theme element which is guaranteed to be translucent? I'll update the patch with the elevating code. Aaron J. Seigo wrote: no such guarantees, i'm afraid, as the themer can in theory screw you over by doing whatever they want but then that's their fault. there is a translucent/ directory and in there are backgrounds that are supposed to be, well, translucent. for clarification: afaiu the translucent folder seems for the blurring case? On topin: maybe stress that translucentbackground.svg really *should* be translucent (not In this case a translucent background looks better - that's entirely subjective) - though it's still a background and no outline. - Thomas --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/#review31550 --- On April 26, 2013, 6 a.m., Martin Gräßlin wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/ --- (Updated April 26, 2013, 6 a.m.) Review request for kwin and Plasma. Description --- Change the visual appearance of the composited outline Uses widgets/translucentbackground as FrameSvg item to ensure that we don't get a huge black square on the screen. When bordering a screen edge we disable the border except if all edges are bordered. This makes a little bit more clear in the quick tiling case what will be the geometry. Blur behind the outline does not work as: * moving the window generates artefacts * moved window needs to be above the outline, but outline needs to be on top of everything. Elevating the moved window results in strange side effects when moving the window below a keep above, etc. Diffs - kwin/geometry.cpp 5a6786d kwin/outline.cpp ad3cfc0 Diff: http://git.reviewboard.kde.org/r/110176/diff/ Testing --- I tried various Plasma theme elements. dialog/* and widgets/background are not useable as there are solid themes. Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 110176: Change the visual appearance of the composited outline
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/#review31539 --- Ship it! seems a nice idea. since the shadow for that item is huge, does take it into account when it resizes the frame? )you would want the frame border, not the shadow more or less aligned with the window size) - Marco Martin On April 25, 2013, 8:58 a.m., Martin Gräßlin wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/ --- (Updated April 25, 2013, 8:58 a.m.) Review request for kwin and Plasma. Description --- Change the visual appearance of the composited outline Uses widgets/translucentbackground as FrameSvg item to ensure that we don't get a huge black square on the screen. When bordering a screen edge we disable the border except if all edges are bordered. This makes a little bit more clear in the quick tiling case what will be the geometry. Blur behind the outline does not work as: * moving the window generates artefacts * moved window needs to be above the outline, but outline needs to be on top of everything. Elevating the moved window results in strange side effects when moving the window below a keep above, etc. Diffs - kwin/outline.cpp ad3cfc01f6fa29010de6845d6c17a54e59d9474e Diff: http://git.reviewboard.kde.org/r/110176/diff/ Testing --- I tried various Plasma theme elements. dialog/* and widgets/background are not useable as there are solid themes. Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 110176: Change the visual appearance of the composited outline
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/#review31550 --- It's a definitively visual improvement with many themes (including and esp. air), but eg. (old) oxygen and several others still get me a black overlay that does not much look like a frame or window and with opaquity it's (surprise) *opaque* (ie. one side of the screen turns -near- black) http://techbase.kde.org/Development/Tutorials/Plasma/ThemeDetails /translucentbackground.svg: a standard background image for plasmoids that for their nature are bigger and with not much text. In this case a translucent background looks better. It needs the same elements of background.svg in it. If this file is not present, the plasmoids that uses this will use background.svg instead. There's (unfortunately) no explicit requirement on that either this is translucent or exists at all (what will get us the regular solid background) For the beginning, i'd strongly recommend to elevate the to-be-tiled window over the outline. - Thomas Lübking On April 25, 2013, 8:58 a.m., Martin Gräßlin wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/ --- (Updated April 25, 2013, 8:58 a.m.) Review request for kwin and Plasma. Description --- Change the visual appearance of the composited outline Uses widgets/translucentbackground as FrameSvg item to ensure that we don't get a huge black square on the screen. When bordering a screen edge we disable the border except if all edges are bordered. This makes a little bit more clear in the quick tiling case what will be the geometry. Blur behind the outline does not work as: * moving the window generates artefacts * moved window needs to be above the outline, but outline needs to be on top of everything. Elevating the moved window results in strange side effects when moving the window below a keep above, etc. Diffs - kwin/outline.cpp ad3cfc01f6fa29010de6845d6c17a54e59d9474e Diff: http://git.reviewboard.kde.org/r/110176/diff/ Testing --- I tried various Plasma theme elements. dialog/* and widgets/background are not useable as there are solid themes. Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 110176: Change the visual appearance of the composited outline
On April 25, 2013, 2:29 p.m., Thomas Lübking wrote: It's a definitively visual improvement with many themes (including and esp. air), but eg. (old) oxygen and several others still get me a black overlay that does not much look like a frame or window and with opaquity it's (surprise) *opaque* (ie. one side of the screen turns -near- black) http://techbase.kde.org/Development/Tutorials/Plasma/ThemeDetails /translucentbackground.svg: a standard background image for plasmoids that for their nature are bigger and with not much text. In this case a translucent background looks better. It needs the same elements of background.svg in it. If this file is not present, the plasmoids that uses this will use background.svg instead. There's (unfortunately) no explicit requirement on that either this is translucent or exists at all (what will get us the regular solid background) For the beginning, i'd strongly recommend to elevate the to-be-tiled window over the outline. @Plasma devs: could you please clarify whether there is a theme element which is guaranteed to be translucent? I'll update the patch with the elevating code. - Martin --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/#review31550 --- On April 25, 2013, 10:58 a.m., Martin Gräßlin wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/ --- (Updated April 25, 2013, 10:58 a.m.) Review request for kwin and Plasma. Description --- Change the visual appearance of the composited outline Uses widgets/translucentbackground as FrameSvg item to ensure that we don't get a huge black square on the screen. When bordering a screen edge we disable the border except if all edges are bordered. This makes a little bit more clear in the quick tiling case what will be the geometry. Blur behind the outline does not work as: * moving the window generates artefacts * moved window needs to be above the outline, but outline needs to be on top of everything. Elevating the moved window results in strange side effects when moving the window below a keep above, etc. Diffs - kwin/outline.cpp ad3cfc01f6fa29010de6845d6c17a54e59d9474e Diff: http://git.reviewboard.kde.org/r/110176/diff/ Testing --- I tried various Plasma theme elements. dialog/* and widgets/background are not useable as there are solid themes. Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 110176: Change the visual appearance of the composited outline
On April 25, 2013, 12:29 p.m., Thomas Lübking wrote: It's a definitively visual improvement with many themes (including and esp. air), but eg. (old) oxygen and several others still get me a black overlay that does not much look like a frame or window and with opaquity it's (surprise) *opaque* (ie. one side of the screen turns -near- black) http://techbase.kde.org/Development/Tutorials/Plasma/ThemeDetails /translucentbackground.svg: a standard background image for plasmoids that for their nature are bigger and with not much text. In this case a translucent background looks better. It needs the same elements of background.svg in it. If this file is not present, the plasmoids that uses this will use background.svg instead. There's (unfortunately) no explicit requirement on that either this is translucent or exists at all (what will get us the regular solid background) For the beginning, i'd strongly recommend to elevate the to-be-tiled window over the outline. Martin Gräßlin wrote: @Plasma devs: could you please clarify whether there is a theme element which is guaranteed to be translucent? I'll update the patch with the elevating code. no such guarantees, i'm afraid, as the themer can in theory screw you over by doing whatever they want but then that's their fault. there is a translucent/ directory and in there are backgrounds that are supposed to be, well, translucent. - Aaron J. --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/#review31550 --- On April 25, 2013, 8:58 a.m., Martin Gräßlin wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110176/ --- (Updated April 25, 2013, 8:58 a.m.) Review request for kwin and Plasma. Description --- Change the visual appearance of the composited outline Uses widgets/translucentbackground as FrameSvg item to ensure that we don't get a huge black square on the screen. When bordering a screen edge we disable the border except if all edges are bordered. This makes a little bit more clear in the quick tiling case what will be the geometry. Blur behind the outline does not work as: * moving the window generates artefacts * moved window needs to be above the outline, but outline needs to be on top of everything. Elevating the moved window results in strange side effects when moving the window below a keep above, etc. Diffs - kwin/outline.cpp ad3cfc01f6fa29010de6845d6c17a54e59d9474e Diff: http://git.reviewboard.kde.org/r/110176/diff/ Testing --- I tried various Plasma theme elements. dialog/* and widgets/background are not useable as there are solid themes. Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel