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

Adds a small field to change the number of sites to display #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

giliam
Copy link

@giliam giliam commented Aug 22, 2019

Simply adds an input field (number type) under the list of websites so you can display more or less websites. I've always found it limited to only display 5 sites.

@supertanuki
Copy link
Collaborator

Hi @giliam and thanks for this contribution.
But I am not sure that adding this feature is useful for everyone.
Before adding a feature, please see https://github.com/supertanuki/Carbonalyser#support--source-code
or let me know by adding an issue.
I let this PR opened. Any other opinions about this?

@giliam
Copy link
Author

giliam commented Sep 12, 2019

Thank you for your answer! It seemed pretty small to me and quite useful to know which websites are consuming too much energy to change my behaviours. Having only 4 sites displayed is kind of limited compared to the total number of sites we visit every day. But I understand :)

@ImaCrea
Copy link

ImaCrea commented Oct 1, 2019

Would be a great addition imho.

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

3 participants