Skip to content

Conversation

@edoardoo
Copy link
Contributor

I've implemented a dark theme feature.
Thought to share it.
Also, feedbacks of any kind are highly appreciated.

@SISheogorath
Copy link
Contributor

Awesome work! Can you provide a screenshot of the dark mode?

Also, could you please fix the syntax so the linter doesn't fail anymore?

All in all I like the idea!

/cc #107

@edoardoo
Copy link
Contributor Author

Sure, I just fixed the linter.
Here is the screenshot:
screenshot-2017-12-28 features - hackmd

@edoardoo
Copy link
Contributor Author

Having now read the #107, if we think that Night mode fits better I'm still in time to change it.
Also, the icon is a moon so it would made more sense.

@ccoenen
Copy link
Contributor

ccoenen commented Dec 29, 2017

One request, judging from the screenshot: I believe the text should not be plain white. Toning it down slightly may look even better.

@edoardoo
Copy link
Contributor Author

edoardoo commented Dec 29, 2017

@ccoenen
For reference: The previous screenshot is with text color #f2f2f2 and headings color: #eee.
This screenshot is with text as #ededed and headings #ddd.
I have to say that you were right, toning the text down is much more eye-relaxing.
I'm gonna push the edit as soon as I can.
screenshot-2017-12-29 features - hackmd

Also, If you want to test it, I've pushed the darkTheme feature to an heroku instance.

In the dev process I've taken as a reference the features page. Do you think it covers extensively the html that could be generated by the markdown?

About the name: maybe Night Mode could be more appealing for the user than Dark mode. So if it's okay for you I would change the name into Night Mode.

@ccoenen
Copy link
Contributor

ccoenen commented Dec 29, 2017

Just looked at it live (thanks!) and I like it a lot!

The Features Page is a very good place to start. I believe it covers most of the things.

Some of the diagrams and the code boxes are not perfect, but I would be okay with merging it regardless. For one thing: this is an improvement, and it does not break anything that has existed before. Also the fine tuning of diagrams / code boxes creates a few simple tickets that may be good starter tickets for new contributors.

@SISheogorath SISheogorath added the feature Wants to add a new feature label Jan 3, 2018
@SISheogorath
Copy link
Contributor

SISheogorath commented Jan 3, 2018

Can you please squash the commits? And sign-off the commit?


How to do?

Squash commits:

git rebase -i HEAD~3
git push -f

Now mark all fix commits at fixup

Sign-off commits afterwards

Some information about Signing off commits can be found in our contribution information:

https://github.com/hackmdio/hackmd/blob/master/CONTRIBUTING.md#sign-your-work

git commit --amend --sign-off

For even more improvements, feel free to GPG sign your work. Information about that:
https://git-scm.com/book/en/v2/Git-Tools-Signing-Your-Work

Signed-off-by: Edoardo Odorico <edoubuntu@gmail.com>
@edoardoo edoardoo changed the title ✨ Implemented dark theme. Implemented dark theme. Jan 4, 2018
@SISheogorath SISheogorath merged commit 7201144 into hackmdio:master Jan 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Wants to add a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants