Skip to content
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

raidboss: added dorgrin's skin as built-in option #5193

Merged
merged 3 commits into from
Feb 1, 2023
Merged

raidboss: added dorgrin's skin as built-in option #5193

merged 3 commits into from
Feb 1, 2023

Conversation

darovic
Copy link
Contributor

@darovic darovic commented Jan 12, 2023

Added my ACT customisations as a built-in skin option. Also included readme with screenshots and custom font instructions.

Thanks for the suggestion @quisquous!

@cactbotbot
Copy link
Collaborator

cactbotbot commented Jan 12, 2023

@darovic Thanks for your contribution! 🌵🚀

@darovic darovic changed the title Added dorgrin's customisations as built-in theme option config: added dorgrin's skin as built-in option Jan 12, 2023
@darovic darovic changed the title config: added dorgrin's skin as built-in option raidboss: added dorgrin's skin as built-in option Jan 12, 2023
@quisquous
Copy link
Owner

Thanks for this! I'm a little distracted by 6.3, but hopefully I will have some time to look soon.


## Peace Sans Font Installation
If you'd like to use the custom font, just follow these instructions. The stylesheet tries to use it; if it's not available, it'll fallback to default.
1. Go to https://www.dafont.com/peace_sans.font
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to include the font directly in the repository and use https://developer.mozilla.org/en-US/docs/Web/CSS/@font-face url to reference it? That way folks don't have to install anything and it will "just work".

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I shall give it a go

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

haha ok, if it's tricky let me know and I'll take a look

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nah should be doable, just so busy this past week :'(

@darovic
Copy link
Contributor Author

darovic commented Feb 1, 2023

All done!

@quisquous quisquous merged commit 477885c into quisquous:main Feb 1, 2023
@quisquous
Copy link
Owner

Thank you!!!

github-actions bot pushed a commit that referenced this pull request Feb 1, 2023
)

Added my [ACT
customisations](https://github.com/darovic/aa-act-customisations) as a
built-in skin option. Also included readme with screenshots and custom
font instructions.

Thanks for the suggestion @quisquous! 477885c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants