----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116027/#review50794 -----------------------------------------------------------
I only have a lowly normal monitor, I haven't tested this visually. src/plasma/framesvg.h <https://git.reviewboard.kde.org/r/116027/#comment35671> willreturn -> will return src/plasma/svg.cpp <https://git.reviewboard.kde.org/r/116027/#comment35673> Do we want to have size and devicePixelRatio separate in the cache ID? It means if I request 20x20 at ratio 1.0, it will have a different ID to 40x40 at ratio 2.0 despite being (as I understand it) visually identical? src/plasma/svg.cpp <https://git.reviewboard.kde.org/r/116027/#comment35675> Don't explain the rationale in the review request and not in the code. People won't be able to find it, and this will end up being code that is confusing. Add your comment //it needs to be integer to have lines contained inside a svg piece to keep being pixel aligned here. - David Edmundson On Feb. 24, 2014, 5:30 p.m., Marco Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116027/ > ----------------------------------------------------------- > > (Updated Feb. 24, 2014, 5:30 p.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > ------- > > This gives support in Svg for devicePixelRatio, just exposed in the api from > Svg side. > > > on the component side, it sets the devicepixelratio of the svg and framesvg > with units. > devicepixelratio is always set integer in the svg, so needs at least 192dpi > to double up. > (it needs to be integer to have lines contained inside a svg piece to keep > being pixel aligned) > > > Diffs > ----- > > src/declarativeimports/core/framesvgitem.h e348c4c > src/declarativeimports/core/framesvgitem.cpp 1826d88 > src/declarativeimports/core/svgitem.h c8be7cc > src/declarativeimports/core/svgitem.cpp e90751a > src/declarativeimports/plasmacomponents/qml/ToolButton.qml 2319879 > src/plasma/framesvg.h 482956d > src/plasma/framesvg.cpp 9e1a53e > src/plasma/private/framesvg_p.h 1bbc744 > src/plasma/private/svg_p.h 332efda > src/plasma/svg.h 01d98f8 > src/plasma/svg.cpp 9ec2aa5 > > Diff: https://git.reviewboard.kde.org/r/116027/diff/ > > > Testing > ------- > > unfortunately i can only test by hardcoding different values and check if the > layout of the widgets stays correct, and is fine, it would be needed somebody > with a retina screen to see it really in action > > > Thanks, > > Marco Martin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
