Merge gtk320 and gtk318 into gtk3
ClosedPublic

Authored by cblack on Aug 1 2019, 3:27 PM.

Details

Summary

GTK 3.20 and GTK 3.18 are now merged into a single GTK 3.0 directory. This will reduce work needed in the future for maintenance and updates.

In the past, GTK3 had an extremely unstable theming API, however, as GTK4 draws near, less focus has been given to breaking existing CSS with GTK3. This makes serving specific versions of GTK CSS redundant and doubles the work required to make changes for gains most users aren't likely to see.

Test Plan

See that themes work properly.

Diff Detail

Repository
R98 Breeze for Gtk
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
cblack created this revision.Aug 1 2019, 3:27 PM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 1 2019, 3:27 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
cblack requested review of this revision.Aug 1 2019, 3:27 PM
ngraham added a subscriber: ngraham.Aug 1 2019, 3:36 PM

Not being an expert in this code or familiar with the history here, it would be helpful if you added a bit more information to the Description section. Specifically, why this split existed in the first place, the function it once served, why it's no longer necessary today, and any visual changes (if any) that result from this patch.

cblack edited the summary of this revision. (Show Details)Aug 1 2019, 8:11 PM
ngraham accepted this revision.Aug 1 2019, 10:53 PM

Thanks!

This revision is now accepted and ready to land.Aug 1 2019, 10:53 PM
ngraham retitled this revision from [Breeze-GTK] Merge gtk320 and gtk318 into gtk3 to Merge gtk320 and gtk318 into gtk3.Aug 2 2019, 4:48 PM
This revision was automatically updated to reflect the committed changes.