-
Notifications
You must be signed in to change notification settings - Fork 88
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document how to theme non-core app assets in app-themes #1338
Comments
@ChrisEdS can you test this and then we can document this together? |
@ChrisEdS, any progress on this one? |
Sorry, completely missed this ticket. When's the next time you'll be in the office? @settermjd |
Should be there on Thursday, @ChrisEdS. |
@VicDeo, it seems that https://doc.owncloud.com/server/10.2/developer_manual/core/theming.html#how-to-override-images already covers this issue? |
@ChrisEdS, does https://doc.owncloud.com/server/10.2/developer_manual/core/theming.html#how-to-override-images already covers this issue? |
@settermjd Currently it has a sample for the core assets only and it's not obvious that theming of apps is also possible
|
Thanks for the further reply, @VicDeo. I'll try and get that implemented. |
What Needs to be Documented?
Theming apps
apps
under your theme:owncloud/apps/theme-example/apps
owncloud/apps/theme-example/apps/files_sharing/img
folder-waiting.svg
intoowncloud/apps/theme-example/apps/files_sharing/img
Note: do not copy the entire app, only the needed assets
Where Does This Need To Be Documented?
https://doc.owncloud.org/server/10.0/developer_manual/core/theming.html
Why Should This Change Be Made? (Optional)
To clarify app-theming for developers
What Type Of Content Change Is This? (Optional)
Which Manual Does This Relate To? (Optional)
Ref owncloud/core#35623
The text was updated successfully, but these errors were encountered: