Page MenuHomeSolus

Initial inclusion for Ant theme fixes
ClosedPublic

Authored by yursan9 on Jan 25 2018, 8:20 AM.
Tags
None
Referenced Files
F11072226: D2076.id.diff
Sat, Aug 12, 2:08 AM
F11053526: D2076.diff
Thu, Aug 10, 11:24 PM
F11015383: D2076.diff
Sat, Aug 5, 10:55 AM
F10957315: D2076.id5012.diff
Mon, Jul 17, 7:12 PM
F10957313: D2076.id.diff
Mon, Jul 17, 7:12 PM
F10949572: D2076.id5064.diff
Sun, Jul 16, 1:52 AM
F10949571: D2076.id5013.diff
Sun, Jul 16, 1:52 AM
F10949569: D2076.id4992.diff
Sun, Jul 16, 1:52 AM
Subscribers

Details

Summary

Initial inclusion for Ant theme fixes T4640

Test Plan

Install the theme, and use it.
Dark theme work too

Diff Detail

Repository
R3989 ant-gtk-theme
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

JoshStrobl added a subscriber: JoshStrobl.
JoshStrobl added inline comments.
package.yml
12–13
15

You should be using install and setting permissions.

16–21

Not sure why you're doing this and not just using a bash for loop to eliminate the redundant lines. Also you don't need the trailing destination in the folder.

This revision now requires changes to proceed.Jan 25 2018, 12:18 PM

Okay @JoshStrobl, I'm sorry. I don't know what kind of syntax accepted by the install process

yursan9 edited edge metadata.

Use for loop for installing file

yursan9 marked an inline comment as done.

Forgot about rundeps

This revision is now accepted and ready to land.Jan 27 2018, 12:43 PM
This revision was automatically updated to reflect the committed changes.