Skip to content

Conversation

TheRedXD
Copy link
Contributor

I've decided to retheme the website (it is now dark mode in true dev fashion)

At the moment it is using assets from a.thered.sh (my assets website), you could replace them though with your own if you don't trust my website lol

I've decided to retheme the website (it is now dark mode in true dev fashion)

At the moment it is using assets from a.thered.sh (my assets website), you could replace them though with your own if you don't trust my website lol
@andrewstech
Copy link

Hi sorry for the extremely late response. I shall speak to my team about it and will keep you posted

@phenax
Copy link
Contributor

phenax commented Aug 6, 2023

This looks good. @is-a-dev/maintainers, what do you think?

@wdhdev
Copy link
Member

wdhdev commented Aug 6, 2023

I like it, but the only thing is the website uses external assets/files.

@phenax
Copy link
Contributor

phenax commented Aug 7, 2023

Good point @WilliamDavidHarrison, it'd be better if the css and the font files were served from here instead.

index.html Outdated
Comment on lines 9 to 11
<link rel="stylesheet" href="https://a.thered.sh/assets/fonts/Roboto/font.css">
<link rel="stylesheet" href="https://a.thered.sh/assets/fonts/UbuntuMono/font.css">
<link rel="stylesheet" href="https://a.thered.sh/assets/fa/css/all.min.css">
Copy link
Contributor

Choose a reason for hiding this comment

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

It'll be better if all font and css files were served from this repo instead.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It should be served from / instead for now, my repo has both fa and fonts there

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Update: no longer served from /, rather I use relative paths which actually work this time after too many commits lol

@wdhdev
Copy link
Member

wdhdev commented Aug 7, 2023

@phenax Can the maintainers team get push access so we can merge it once it is finished if you're not online?

@TheRedXD
Copy link
Contributor Author

TheRedXD commented Sep 3, 2023

I'll be fixing this up, the fa/css/all.min.css is a css file linking to fontawesome, you need to include fontawesome in your thing to get it working, the fonts are pretty easy to import, I can set everything up, just give me a bit...

TheRedXD and others added 2 commits September 3, 2023 03:27
in the process of moving assets to repository instead of a.thered.sh
@TheRedXD TheRedXD requested a review from phenax September 3, 2023 00:55
@TheRedXD
Copy link
Contributor Author

TheRedXD commented Sep 3, 2023

i fixed it, there's also a deployed demo https://theredxd.github.io/is-a-dev.github.io/

@wdhdev
Copy link
Member

wdhdev commented Sep 3, 2023

Is Font Awesome hosted on CDNJS? That would be better than hosting it ourselves due to the amount of lines.

@TheRedXD
Copy link
Contributor Author

TheRedXD commented Sep 3, 2023

Looks like it!
https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.4.2/css/all.min.css
I can update my instance of the site with this version

@TheRedXD
Copy link
Contributor Author

TheRedXD commented Sep 3, 2023

Copy link
Member

Choose a reason for hiding this comment

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

Please remove this file as it is not needed

Copy link
Member

Choose a reason for hiding this comment

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

Please remove this file as it is not needed

Copy link
Member

Choose a reason for hiding this comment

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

Please remove this file as it is not needed

</main>
</body>
</body>

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is licensing, generally you'd want to leave that in but ig I can remove it?

Copy link
Member

Choose a reason for hiding this comment

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

@TheRedXD Half of the licenses are just blank and also they aren't referenced anywhere either. We can always add attribution in the footer.

Copy link
Member

@wdhdev wdhdev left a comment

Choose a reason for hiding this comment

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

Some of these fonts are available on Google Fonts, would we be able to use those instead of hosting it ourselves?

@TheRedXD
Copy link
Contributor Author

TheRedXD commented Sep 5, 2023

Some of these fonts are available on Google Fonts, would we be able to use those instead of hosting it ourselves?

Yes, I was asked to have it serve from the repo though

Copy link
Member

@wdhdev wdhdev left a comment

Choose a reason for hiding this comment

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

👌 LGTM!
@phenax

@phenax phenax merged commit 129db40 into is-a-dev:main Sep 10, 2023
@phenax
Copy link
Contributor

phenax commented Sep 10, 2023

@WilliamDavidHarrison Sorry for the delay, I've given write access to the maintainers team.

@wdhdev
Copy link
Member

wdhdev commented Sep 10, 2023

@phenax All good. By the way, is there a faster way to contact you if needed like Telegram or something? Also, can you give them the top maintainer role on Discord?

@phenax
Copy link
Contributor

phenax commented Sep 10, 2023

@WilliamDavidHarrison, email would be the most convinient. If it is something urgent, please mention [URGENT] in the subject and I'll try to get it to it sooner.

orangci pushed a commit that referenced this pull request Apr 9, 2025
I rethemed the website a bit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants