Re: Review Request 126738: Remove external dependencies from svgs
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126738/ --- (Updated July 6, 2016, 12:12 p.m.) Status -- This change has been discarded. Review request for KDE Frameworks, Andreas Kainz and Uri Herrera. Repository: breeze-icons Description --- I don't know what I'm doing here. But when using the icons in a QML app I get a lot of warnings like this one: Could not resolve property : linearGradient4548 Running the svgs through inkscape and using the "clean up document" function results in this commit (and the warnings are gone). This may not be the right thing to do but it would be nice to get rid of all these warnings when using the icons. Signed-off-by: Dirk Hohndel Diffs - icons/actions/24/dialog-cancel.svg a2b022a9dc7b7cf97e4c3391b87e7800355d2dcb icons/actions/24/distribute-horizontal-x.svg 262fcaca937cb72d8393fd5dff9c59651367fe36 icons/actions/24/document-edit.svg afb37ca9624b37d6ac5aa9c94f1b2cef620faff6 icons/actions/24/document-save.svg cee0a3521deb7eb1fcb79266afaa8951f4984b20 Diff: https://git.reviewboard.kde.org/r/126738/diff/ Testing --- Thanks, Dirk Hohndel ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 126738: Remove external dependencies from svgs
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126738/#review96341 --- I made a cleanup off all breeze icons so it should now work without troubles if you find some please let me know. otherwise you can close the request. thanks Andreas - Andreas Kainz On Jan. 14, 2016, 1:18 vorm., Dirk Hohndel wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126738/ > --- > > (Updated Jan. 14, 2016, 1:18 vorm.) > > > Review request for KDE Frameworks, Andreas Kainz and Uri Herrera. > > > Repository: breeze-icons > > > Description > --- > > I don't know what I'm doing here. But when using the icons in a QML app I > get a lot of warnings like this one: > > Could not resolve property : linearGradient4548 > > Running the svgs through inkscape and using the "clean up document" > function results in this commit (and the warnings are gone). > > This may not be the right thing to do but it would be nice to get rid of > all these warnings when using the icons. > > Signed-off-by: Dirk Hohndel > > > Diffs > - > > icons/actions/24/dialog-cancel.svg a2b022a9dc7b7cf97e4c3391b87e7800355d2dcb > icons/actions/24/distribute-horizontal-x.svg > 262fcaca937cb72d8393fd5dff9c59651367fe36 > icons/actions/24/document-edit.svg afb37ca9624b37d6ac5aa9c94f1b2cef620faff6 > icons/actions/24/document-save.svg cee0a3521deb7eb1fcb79266afaa8951f4984b20 > > Diff: https://git.reviewboard.kde.org/r/126738/diff/ > > > Testing > --- > > > Thanks, > > Dirk Hohndel > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 126738: Remove external dependencies from svgs
> On Jan. 14, 2016, 5:51 a.m., Dirk Hohndel wrote: > > this will need to be done for dozens more SVGs - I sent the four that I > > needed for my app for review, but if this change gets accepted I'll > > volunteer to rewrite all of the SVGs that contain external references. It > > just seemed silly to spend hours doing that and then have the project > > reject the change :-) > > andreas kainz wrote: > what you need can it be included in > https://git.reviewboard.kde.org/r/126740/ than we may close this request? My RR doesn't cover this I'm afraid. Also I'm not too sure how fast will be to end up adopting this. I suggest to get this patch in (or a more complete iteration of it with the rest of svgs if that's possible). - Aleix --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126738/#review91047 --- On Jan. 14, 2016, 2:18 a.m., Dirk Hohndel wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126738/ > --- > > (Updated Jan. 14, 2016, 2:18 a.m.) > > > Review request for KDE Frameworks, andreas kainz and Uri Herrera. > > > Repository: breeze-icons > > > Description > --- > > I don't know what I'm doing here. But when using the icons in a QML app I > get a lot of warnings like this one: > > Could not resolve property : linearGradient4548 > > Running the svgs through inkscape and using the "clean up document" > function results in this commit (and the warnings are gone). > > This may not be the right thing to do but it would be nice to get rid of > all these warnings when using the icons. > > Signed-off-by: Dirk Hohndel > > > Diffs > - > > icons/actions/24/dialog-cancel.svg a2b022a9dc7b7cf97e4c3391b87e7800355d2dcb > icons/actions/24/distribute-horizontal-x.svg > 262fcaca937cb72d8393fd5dff9c59651367fe36 > icons/actions/24/document-edit.svg afb37ca9624b37d6ac5aa9c94f1b2cef620faff6 > icons/actions/24/document-save.svg cee0a3521deb7eb1fcb79266afaa8951f4984b20 > > Diff: https://git.reviewboard.kde.org/r/126738/diff/ > > > Testing > --- > > > Thanks, > > Dirk Hohndel > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 126738: Remove external dependencies from svgs
> On Jan. 14, 2016, 4:51 vorm., Dirk Hohndel wrote: > > this will need to be done for dozens more SVGs - I sent the four that I > > needed for my app for review, but if this change gets accepted I'll > > volunteer to rewrite all of the SVGs that contain external references. It > > just seemed silly to spend hours doing that and then have the project > > reject the change :-) what you need can it be included in https://git.reviewboard.kde.org/r/126740/ than we may close this request? - andreas --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126738/#review91047 --- On Jan. 14, 2016, 1:18 vorm., Dirk Hohndel wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126738/ > --- > > (Updated Jan. 14, 2016, 1:18 vorm.) > > > Review request for KDE Frameworks, andreas kainz and Uri Herrera. > > > Repository: breeze-icons > > > Description > --- > > I don't know what I'm doing here. But when using the icons in a QML app I > get a lot of warnings like this one: > > Could not resolve property : linearGradient4548 > > Running the svgs through inkscape and using the "clean up document" > function results in this commit (and the warnings are gone). > > This may not be the right thing to do but it would be nice to get rid of > all these warnings when using the icons. > > Signed-off-by: Dirk Hohndel > > > Diffs > - > > icons/actions/24/dialog-cancel.svg a2b022a9dc7b7cf97e4c3391b87e7800355d2dcb > icons/actions/24/distribute-horizontal-x.svg > 262fcaca937cb72d8393fd5dff9c59651367fe36 > icons/actions/24/document-edit.svg afb37ca9624b37d6ac5aa9c94f1b2cef620faff6 > icons/actions/24/document-save.svg cee0a3521deb7eb1fcb79266afaa8951f4984b20 > > Diff: https://git.reviewboard.kde.org/r/126738/diff/ > > > Testing > --- > > > Thanks, > > Dirk Hohndel > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 126738: Remove external dependencies from svgs
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126738/#review91047 --- this will need to be done for dozens more SVGs - I sent the four that I needed for my app for review, but if this change gets accepted I'll volunteer to rewrite all of the SVGs that contain external references. It just seemed silly to spend hours doing that and then have the project reject the change :-) - Dirk Hohndel On Jan. 14, 2016, 1:18 a.m., Dirk Hohndel wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126738/ > --- > > (Updated Jan. 14, 2016, 1:18 a.m.) > > > Review request for KDE Frameworks, andreas kainz and Uri Herrera. > > > Repository: breeze-icons > > > Description > --- > > I don't know what I'm doing here. But when using the icons in a QML app I > get a lot of warnings like this one: > > Could not resolve property : linearGradient4548 > > Running the svgs through inkscape and using the "clean up document" > function results in this commit (and the warnings are gone). > > This may not be the right thing to do but it would be nice to get rid of > all these warnings when using the icons. > > Signed-off-by: Dirk Hohndel > > > Diffs > - > > icons/actions/24/dialog-cancel.svg a2b022a9dc7b7cf97e4c3391b87e7800355d2dcb > icons/actions/24/distribute-horizontal-x.svg > 262fcaca937cb72d8393fd5dff9c59651367fe36 > icons/actions/24/document-edit.svg afb37ca9624b37d6ac5aa9c94f1b2cef620faff6 > icons/actions/24/document-save.svg cee0a3521deb7eb1fcb79266afaa8951f4984b20 > > Diff: https://git.reviewboard.kde.org/r/126738/diff/ > > > Testing > --- > > > Thanks, > > Dirk Hohndel > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 126738: Remove external dependencies from svgs
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126738/#review91043 --- +1 Good catch. - Aleix Pol Gonzalez On Jan. 14, 2016, 2:18 a.m., Dirk Hohndel wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126738/ > --- > > (Updated Jan. 14, 2016, 2:18 a.m.) > > > Review request for KDE Frameworks, andreas kainz and Uri Herrera. > > > Repository: breeze-icons > > > Description > --- > > I don't know what I'm doing here. But when using the icons in a QML app I > get a lot of warnings like this one: > > Could not resolve property : linearGradient4548 > > Running the svgs through inkscape and using the "clean up document" > function results in this commit (and the warnings are gone). > > This may not be the right thing to do but it would be nice to get rid of > all these warnings when using the icons. > > Signed-off-by: Dirk Hohndel > > > Diffs > - > > icons/actions/24/dialog-cancel.svg a2b022a9dc7b7cf97e4c3391b87e7800355d2dcb > icons/actions/24/distribute-horizontal-x.svg > 262fcaca937cb72d8393fd5dff9c59651367fe36 > icons/actions/24/document-edit.svg afb37ca9624b37d6ac5aa9c94f1b2cef620faff6 > icons/actions/24/document-save.svg cee0a3521deb7eb1fcb79266afaa8951f4984b20 > > Diff: https://git.reviewboard.kde.org/r/126738/diff/ > > > Testing > --- > > > Thanks, > > Dirk Hohndel > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Review Request 126738: Remove external dependencies from svgs
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126738/ --- Review request for KDE Frameworks. Repository: breeze-icons Description --- I don't know what I'm doing here. But when using the icons in a QML app I get a lot of warnings like this one: Could not resolve property : linearGradient4548 Running the svgs through inkscape and using the "clean up document" function results in this commit (and the warnings are gone). This may not be the right thing to do but it would be nice to get rid of all these warnings when using the icons. Signed-off-by: Dirk Hohndel Diffs - icons/actions/24/dialog-cancel.svg a2b022a9dc7b7cf97e4c3391b87e7800355d2dcb icons/actions/24/distribute-horizontal-x.svg 262fcaca937cb72d8393fd5dff9c59651367fe36 icons/actions/24/document-edit.svg afb37ca9624b37d6ac5aa9c94f1b2cef620faff6 icons/actions/24/document-save.svg cee0a3521deb7eb1fcb79266afaa8951f4984b20 Diff: https://git.reviewboard.kde.org/r/126738/diff/ Testing --- Thanks, Dirk Hohndel ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel