Document & simplify deployment as serverwide Gitea theme #1

Merged
Freeplay merged 3 commits from norwin/gitea-modern-theme:main into main 11 months ago
norwin commented 11 months ago

The theme looks great so far, thank you!

I went ahead and

  • changed the naming scheme, so inclusion as builtin Gitea theme is simplified
  • documented how to include the theme with Gitea
  • fixed a syntax error in the LESS file (but didn't recompile as I don't have the tooling installed)
The theme looks great so far, thank you! I went ahead and - changed the naming scheme, so inclusion as builtin Gitea theme is simplified - documented how to include the theme with Gitea - fixed a syntax error in the LESS file (but didn't recompile as I don't have the tooling installed)
norwin added 2 commits 11 months ago
norwin reviewed 11 months ago
When making contributions, don't use the compiled version.
1. Download the [`theme-gitea-modern.css` file][theme-file] and add it to your custom folder in `$GITEA_CUSTOM/public/css/` ([learn how to find that directory][doc-dir]).
- The path has changed in Gitea 1.15, there the file needs to be placed in `$GITEA_CUSTOM/public/assets/css`
Poster

I didn't verify that this new path is correct so far, it may be wrong.

I didn't verify that this new path is correct so far, it may be wrong.
Owner

I would actually prefer to keep it as the Stylus preprocessor, as that is what I'm most familiar with. I saw that it also worked with Less and assuming other preprocessors that support CSS nesting as well (I didn't use much of other Stylus features), so I decided to give people that option.
I probably should have also mentioned that you would need to change the file extension to whatever preprocessor you use.

I would actually prefer to keep it as the Stylus preprocessor, as that is what I'm most familiar with. I saw that it also worked with Less and assuming other preprocessors that support CSS nesting as well (I didn't use much of other Stylus features), so I decided to give people that option. I probably should have also mentioned that you would need to change the file extension to whatever preprocessor you use.
Poster

Ah ok, yeah we can name the source file differently than my suggestion. Any suggestion?

Ah ok, yeah we can name the source file differently than my suggestion. Any suggestion?
Owner

I might just keep it as Stylus, which the file extension for that is .styl

I might just keep it as Stylus, which the file extension for that is `.styl`
norwin added 1 commit 11 months ago
Owner

The rest I think looks good, I'll merge

The rest _I think_ looks good, I'll merge
Freeplay merged commit 24c1624a5d into main 11 months ago
The pull request has been merged as 24c1624a5d.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.