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

Name & logo #18

Merged
merged 13 commits into from
Nov 16, 2020
Merged

Name & logo #18

merged 13 commits into from
Nov 16, 2020

Conversation

nrslt
Copy link
Contributor

@nrslt nrslt commented Nov 11, 2020

  • added a app.title in main.py
  • created an assets/ directory that stores the favicon.ico file, necessary to generate a browser tab icon

Screenshot 2020-11-11 at 17 25 07

nrslt and others added 11 commits October 12, 2020 13:00
getting up to date with original repo
feat: Set up dockerized server for the dash app (#3)
…jects to be used in the homepage.py file ; changed the layout of the homepage in order to better fit our needs
feat: Sets the project for automatic deployment on Heroku (#5)
graphs.py new file + some changes in homepage.py layout ! (#7)
Added the slider and the callback for interactivity (#8)
@nrslt nrslt requested a review from frgfm November 11, 2020 16:11
@frgfm frgfm added the type: enhancement New feature or request label Nov 11, 2020
@frgfm frgfm added this to the 0.1.0 milestone Nov 11, 2020
@frgfm frgfm linked an issue Nov 11, 2020 that may be closed by this pull request
@frgfm
Copy link
Member

frgfm commented Nov 11, 2020

@nrslt Thanks for the PR! Would you mind sharing a screenshot of how it renders? (so that we can keep track of evolutions when scrolling through merged PRs)

Copy link
Member

@frgfm frgfm left a comment

Choose a reason for hiding this comment

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

Thanks again! I added some comments. Also, could you check if you find the proper icons over here : https://github.com/pyronear/pyronear.github.io/tree/master/img
If so, it would remove the need to add some content in app/assets!

app/main.py Outdated Show resolved Hide resolved
@nrslt
Copy link
Contributor Author

nrslt commented Nov 11, 2020

I changed the .ico to put the right one and also changed the title to "Pyronear - Monitoring platform"
the screenshot is in the PR description above!

@frgfm
Copy link
Member

frgfm commented Nov 11, 2020

@nrslt Thanks a lot!
My bad my previous comment wasn't very clear: is it possible to refer to the distant favicon by URL (https://github.com/pyronear/pyronear.github.io/blob/master/img/favicon.ico)? rather than committing it here?

@nrslt
Copy link
Contributor Author

nrslt commented Nov 15, 2020

@frgfm from what I've read so far, it seems to be only possible to change the favicon by adding a .ico file in the 'assets' folder that is at the root of the app..
I've tried to find other solutions but at this point I do not know how to use a distant url to specify the favicon file!

@frgfm
Copy link
Member

frgfm commented Nov 15, 2020

Thanks for investigating @nrslt ! Let's merge the PR then 👌

Copy link
Member

@frgfm frgfm left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

@nrslt
Copy link
Contributor Author

nrslt commented Nov 16, 2020

great can you merge it, then? :)

@frgfm frgfm merged commit c6a5aa6 into pyronear:master Nov 16, 2020
@frgfm
Copy link
Member

frgfm commented Nov 16, 2020

Yes sure! fyi, when you're the author of the PR and you have writing access to the repo, it's better if you do it :) (in case things go south, you're the most appropriate person to figure out what could have happened)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Updating HTML page title & logo
2 participants