> On 2010-12-05 14:50:33, Manuel Mommertz wrote: > > It is now even worse then before. Unthemed graphics still use the colors > > from plasma theme but don't get informed if the theme changes. this means, > > the colors that are used when first rendering the graphics stay even if I > > switch the theme. > > The plasma elements itself don't get rerendered on theme change. Only > > graphics that have to be rerendered for another reason (resizing a widget, > > ...) get the svg's from the new activated theme. > > Marco Martin wrote: > well, they shouldn't be informed on theme changes in this case... > > Manuel Mommertz wrote: > right, but on colorscheme changes. with doesn't happen. > > Aaron Seigo wrote: > "Unthemed graphics still use the colors from plasma theme but don't get > informed if the theme changes." > > unthemed graphics never use colors from the plasma theme. they either use > no colors, or they use the system colors. in which case are unthemed graphics > using the color scheme from the plasma theme? > > "the colors that are used when first rendering the graphics stay even if > I switch the theme." > > of course. the theme is not relevant. you get to pick between "themed" > and "unthemed". > > "The plasma elements itself don't get rerendered on theme change." > > which plasma elements? > > "Only graphics that have to be rerendered for another reason (resizing a > widget, ...) get the svg's from the new activated theme." > > are you saying that NO svg's (themed or unthemed) are being updated? > because that works here. > > "right, but on colorscheme changes. with doesn't happen." > > do you mean the KDE color scheme change? e.g. when you change the > colorscheme in the Colors control panel? > > i'm quite confused now by what you mean. can you give me a clear, simple > use case with detailed steps of reproduction so that i may identify and > reproduce the problem and subsequently fix it? thanks. > > Manuel Mommertz wrote: > http://img502.imageshack.us/img502/2729/plasmadesktopxk7517.png > > Steps I did before taking this screenshot: > > Logged in while Oxygen was the selected theme. > Switched theme to Air. > > Noticed: After switching all graphics stay unchanged. The folderview > changed after some 10-20 seconds. The handle (seen next to folderview) > changed to air after I unhovered folderview and hovered it again. Panel stays > unchanged. The Entry in the taskbar gets the hover-graphic from air while > falling back to oxygen when mouse leaves the entry. > Windeco still uses oxywin colors. > > What I did to ensure this is not some strange local thing: > Logged out > Removed cache files from themes (oxygen and air, each two files. No other > files were there from plasma) > Removed kde-install-path > Reinstalled > Logged in again > Nothing changed > > What I am going to do now: > rebuild kdelibs and kdebase and test again
none of this had anything to do with these changes, however. these were all little bugs that had crept into the codebase around repainting when the theme changes. i've fixed the ones i've seen so far already. but again, nothing to do with these changes. they would have been broken before this commit, too. what is broken (so i can reproduce, test and fix) with unthemed svgs? - Aaron ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6008/#review9136 ----------------------------------------------------------- On 2010-11-28 23:52:16, Aaron Seigo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/6008/ > ----------------------------------------------------------- > > (Updated 2010-11-28 23:52:16) > > > Review request for Plasma. > > > Summary > ------- > > see: http://www.mail-archive.com/plasma-devel@kde.org/msg13494.html > > this introduces a Theme object used by Svg to cache the rendering of svg's > asking to be colorized, but which are not part of the theme. this requires an > independent cache and set of color files, therefore a second Theme object. > SvgPrivate::actualTheme() is thus overridden by > SvgPrivate::cacheAndColorsTheme() whenever caching or colors are used. > > there are two remaining defects of varying severity, though neither is very > high imho: > > a) if the system color palette changes when the application is not running > (or an svg using such a theme is not running) the cache will not be dropped. > this is an existing issue, however, not something introduce by this patchset > > b) the new system color theme is shared for performance reasons, but once > created it is never deleted. it ought to be reference counted so it can be > cleaned up after in use, but this is currently not done. > > several other fixes crept into this patchset (sorry :/) including: > > * not dropping the cache just because we change themes; this made sense when > just plasma-desktop used this or when the apps using Plasma::Theme all used > the same theme. this is no longer the case, really, and having applications > changing themes randomly kicking the cache out from underneath other apps > that may still be using it is undesirable. this does mean that cache files > will accumulate. not sure that's a big issue as they don't tend to be large > and are in an area marked as "cache" (so ripe for clean-without-care) > > * consolidate the signal/slot connections in SvgPrivate::checkColorHints, and > now the check is consistent on all paths (previously, it wasn't!) > > * missing (and possible cause of cache key ambiguity) separator in > CACHE_ID_WITH_SIZE > > * cleanups such as getting rid of superfluous use of Plasma:: namespacing in > code that is now in libplasma, such as the stylesheeting. (it was from a > plasmoid sebas wrote originally, iirc, explaining the historical reason for > the explicit namespacing) > > in any case, consider this a draft at this stage. i'm interested in geting > feedback, improvements and testing. > > > Diffs > ----- > > /trunk/KDE/kdelibs/plasma/private/svg_p.h 1199366 > /trunk/KDE/kdelibs/plasma/svg.cpp 1201684 > /trunk/KDE/kdelibs/plasma/theme.cpp 1201685 > > Diff: http://svn.reviewboard.kde.org/r/6008/diff > > > Testing > ------- > > just running normal plasma-desktop and what i use. needs testing with svg's > that will actually rely on this feature. > > > Thanks, > > Aaron > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
