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 background color timestamp #128

Closed
op3nus3r opened this Issue Jan 13, 2018 · 5 comments

Comments

Projects
None yet
2 participants
@op3nus3r
Copy link

op3nus3r commented Jan 13, 2018

Hello everyone, thank you for this excellent application which I use with great pleasure on my Raspberry Pi 3.
I use timestamping to mark captured images, during the day the image is white and the timestamping is written in black, everything is fine.
During the night the image is black and the timestamp is always black, so you don't see it.
Is it possible to add a color background to the timestamp?

@cedricve

This comment has been minimized.

Copy link
Member

cedricve commented Jan 14, 2018

hey @op3nus3r, what I usually do is setting the timestamp color to red..

@op3nus3r

This comment has been minimized.

Copy link

op3nus3r commented Jan 14, 2018

Hello @cedricve , thank you for your answer.
That's what I also do, I added the red for the time stamp. I tried a little bit of every color. Under certain conditions, when the sun sets on a white wall or with yellow public lighting, the red timestamp is difficult to read as well.
It was just a suggestion to be able to configure a background color for timestamping if needed, it's not essential I thought I might not be the only one in this situation.

@cedricve

This comment has been minimized.

Copy link
Member

cedricve commented Jan 14, 2018

makes sense @op3nus3r, I think we'll add a background color by default then.. Let me assign a label to it, and add it to our next release.

@cedricve cedricve changed the title Add background timestamp Add background color timestamp Jan 14, 2018

@cedricve cedricve self-assigned this Jan 14, 2018

@op3nus3r

This comment has been minimized.

Copy link

op3nus3r commented Jan 17, 2018

Thank you very much for your answers, I don't doubt that this will be useful to a lot of people.

cedricve added a commit that referenced this issue Feb 1, 2018

@cedricve

This comment has been minimized.

Copy link
Member

cedricve commented Feb 1, 2018

will be available in next release @op3nus3r, thanks for sharing!

@cedricve cedricve closed this Feb 1, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment