Major overhaul of Urchin #97
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Added TailwindCSS to Urchin. - Updated Makefile to download TailwindCSS (currently only Linux is supported) - Refactored templates to share similar logic between templates. - Redesigned UI for Urchin. - Added layout page shared between all sub pages. - Added dark mode support. - Added new `renderHtml` method to reduce duplicate code in go. - Added self-designed minimalistic 'sea urchin' favicon. :D - Added missing sites `about` and `services`. - Added `Not Found` handling if a resource is requested that is not present, displaying an error message while keeping the layout page. - Restructured `static` folder to include stylesheets, scripts and assets.
- Updated Makefile to use `wget` instead of `cURL` to download TailwindCSS CLI. - Removed `cURL` from Dockerfile installation.
|
Hey @AlDu2407 , I wonder if you ever got a chance to add Btw I think it has |
|
Hi @matheusgomes28 , is there a way for you to remove the line in the Dockerfile, that removes wget from the build image? I think afterwards the pipeline will succeed. And afterwards you should be able to review the entire code. |
|
|
||
| import "github.com/matheusgomes28/urchin/common" | ||
|
|
||
| templ makeNotFound() { |
There was a problem hiding this comment.
Maybe we show the requested resource in the header text
There was a problem hiding this comment.
can do later, not an issue
|
|
||
| templ MakeNotFoundPage(links []common.Link) { | ||
| @MakeLayout("Not Found", links, makeNotFound()) | ||
| } |
There was a problem hiding this comment.
New line all of these files at the end
+ Make sure we download a particular version of tailwind. + Make sure we do a checksum on the binary.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
renderHtmlmethod to reduce duplicate code in go.aboutandservices.Not Foundhandling if a resource is requested that is not present, displaying an error message while keeping the layout page.staticfolder to include stylesheets, scripts and assets.