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

add TRIfA to website #238

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

add TRIfA to website #238

wants to merge 4 commits into from

Conversation

zoff99
Copy link

@zoff99 zoff99 commented Dec 2, 2021

No description provided.

@nurupo
Copy link
Member

nurupo commented Dec 13, 2021

themes/website/static/js/osdetect.js and themes/website/templates/clients.html are missing entries for TRIfA.

When adding client screenshots, please run them through:

find . -type f -iname "*.png" -print0 | xargs -I {} -0 optipng -o7 -zm1-9 "{}"
find . -type f -iname "*.jp*" -print0 | xargs -I {} -0 jpegoptim --strip-all "{}"

(don't run it on the screenshots already on the website).

@zoff99
Copy link
Author

zoff99 commented Dec 16, 2021

@nurupo @iphydf --> done.
should i squash?

@zoff99
Copy link
Author

zoff99 commented Dec 20, 2021

any news on this?

correct the wording,
add trifa to js,
add trifa screenshots,
add warning about push notification server access.
@nurupo
Copy link
Member

nurupo commented Dec 28, 2021

How about something like this (the three commits I have added)?

trifa_2.png was removed because it wasn't a screenshot.

@nurupo
Copy link
Member

nurupo commented Jan 5, 2022

Someone pointed out that osdetect.js would add trifa download to the top of the download page without any warnings shown, so something should be done to remedy this.

@zoff99
Copy link
Author

zoff99 commented Feb 26, 2022

is there any progress on this?

@nurupo
Copy link
Member

nurupo commented Feb 26, 2022

Sorry, I'm away for a few months, maybe @robinlinden or someone else can take over this.

@zoff99
Copy link
Author

zoff99 commented Oct 10, 2022

any news on this? is this going to be merged?

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