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

Add favicon to Playroom site #309

Merged
merged 9 commits into from Dec 18, 2023
Merged

Add favicon to Playroom site #309

merged 9 commits into from Dec 18, 2023

Conversation

felixhabib
Copy link
Contributor

@felixhabib felixhabib commented Dec 15, 2023

This was a little hackathon project.

  • Adding a favicon to playroom so you can tell playroom tabs apart if the tab title gets cut off.
  • The favicon should change colour appropriately based on you browser theme (dark/light mode).
  • This uses the current logo from the README file but happy to discuss if something else is more appropriate.

Copy link

changeset-bot bot commented Dec 15, 2023

🦋 Changeset detected

Latest commit: 14678c6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
playroom Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@felixhabib felixhabib changed the title Add favicon Add favicon to Playroom site Dec 15, 2023
@felixhabib felixhabib marked this pull request as ready for review December 15, 2023 04:49
@felixhabib felixhabib requested a review from a team as a code owner December 15, 2023 04:49
@felixhabib felixhabib marked this pull request as draft December 15, 2023 04:51
@felixhabib felixhabib marked this pull request as ready for review December 15, 2023 05:08
@felixhabib felixhabib marked this pull request as draft December 15, 2023 05:11
felixhabib and others added 2 commits December 18, 2023 10:57
Co-authored-by: Remus Mate <3297808+mrm007@users.noreply.github.com>
@felixhabib felixhabib marked this pull request as ready for review December 18, 2023 00:59
@felixhabib felixhabib merged commit e3b820b into master Dec 18, 2023
6 checks passed
@felixhabib felixhabib deleted the add-favicon branch December 18, 2023 01:01
@@ -163,6 +167,7 @@ module.exports = async (playroomConfig, options) => {
chunksSortMode: 'none',
chunks: ['index'],
filename: 'index.html',
favicon: 'images/favicon.png',
Copy link

Choose a reason for hiding this comment

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

Could this have come from playroomConfig? Could this be coming from a public folder too. But offering it in the playroomConfig would make it more flexible.

Currently getting an error when building playroom.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey David, could you log the error as an Issue with a reproduction? Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm getting an error too:

Error: HtmlWebpackPlugin: could not load file /<path-to-project>/images/favicon.png

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.

None yet

5 participants