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

Suggestion: Color Themes #58

Open
CorneliousJD opened this issue Jan 26, 2022 · 12 comments · May be fixed by #182
Open

Suggestion: Color Themes #58

CorneliousJD opened this issue Jan 26, 2022 · 12 comments · May be fixed by #182

Comments

@CorneliousJD
Copy link

I'd love a way to set a docker container variable to change the color of certain elements, or possibly even to add a logo somewhere on the page. I'd love to use this for work to request password data from clients.

Right now we are using a branded version of onetimesecret with our primary color, a favicon, and our logo, but it doesn't play very nice on a mobile phone.

If I could replicate our use case for that and set this up instead, I think that would be easier/better for us, but we'd want it branded and to have our logo in it.

Or perhaps we could just embed this via an iFrame on on our website instead?

PS - How can we edit the templates via Docker install?

@AndrewPaglusch
Copy link
Owner

I've come across quite a few deployments of FlashPaper on the internet with logos added in, so this is definitely a feature others want as well. I will take a look into getting this implemented. Thanks!

As for adding in new templates: You need to map a volume to /var/www/templates in the container. Only files ending in ".txt" will be listed.

volumes:
  - './data:/var/www/html/data'
  - './templates:/var/www/templates'

@CorneliousJD
Copy link
Author

Thanks for the consideration, the way I guess I envision it within docker would be one of two ways.

1, inside the /data volume you could have an "images" folder with a "logo.png" and a "favicon.ico" that users could override with their own files.
or 2, just have variables for LOGO and FAVICON that you can point to resources hosted elsewhere. (this may be cleaner to implement?)
And then perhaps a variable for PRIMARYCOLOR or something like that which accepts a HEX color code? That could just change the color of the button, and a SECONDARYCOLOR that could change the color of the grey background box.

@Bulletoverload
Copy link

Seconded. I would also like to use this for my personal business and the ability to add logos/color themes would be super cool.

Also, how about sub-headers for message view_secret and confirm_view_secret?

@trulow
Copy link

trulow commented Apr 18, 2024

Any update to this request?

A dark theme and an option to add a logo would be amazing!

@AndrewPaglusch
Copy link
Owner

Hey @trulow thanks for reaching out! There hasn't been any movement on implementing this yet. This is something I'd like to see added some day, but it's not a priority right now and more of a nice-to-have feature. If you'd like to take a shot at adding, we'd be happy to review a PR. Otherwise, please just keep an eye on this issue.

@krumka
Copy link

krumka commented May 22, 2024

Hi, that would be very awesome to have colors and logos !

I've seen that a PR was created for this feature (thanks @barry-smithjr) (at least dark mode even if no custom colors)
@AndrewPaglusch any idea on when/if you would merge this PR ?

Thanks for this awesome project anyway !

@AndrewPaglusch
Copy link
Owner

Hey @krumka. A lot has been happening in my personal life over the past two weeks, so I haven't been able to give the PR a good review yet. I plan on reviewing it this weekend. Keep an eye out for more updates!

@dotziph
Copy link

dotziph commented Oct 19, 2024

Any updates on this?

@AndrewPaglusch
Copy link
Owner

Hey @dotziph. We're still discussing this internally. An update will be posted to this request as soon as we have new info to share. Thanks!

@barry-smithjr
Copy link
Contributor

Quick update:

Working on adding a custom colors section and support for Bootswatch themes.

@barry-smithjr barry-smithjr linked a pull request Oct 26, 2024 that will close this issue
@AndrewPaglusch
Copy link
Owner

Another update: @barry-smithjr and I are working to review the PR he submitted. I hope to have this approved and merged as soon as possible

@AndrewPaglusch
Copy link
Owner

@barry-smithjr and I spent multiple hours working on this PR tonight. We've run into quite a few issues, but we're working on it. Stay tuned!

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 a pull request may close this issue.

7 participants