Skip to content

docs: update README.md to link banner to website#99

Open
dalathegreat wants to merge 2 commits intofluxerapp:canaryfrom
dalathegreat:patch-1
Open

docs: update README.md to link banner to website#99
dalathegreat wants to merge 2 commits intofluxerapp:canaryfrom
dalathegreat:patch-1

Conversation

@dalathegreat
Copy link

Summary

This PR updates the README.md image with a link

  • What: We make the banner logo contain a hyperlink towards https://fluxer.app/
  • Why: It is intuitive for users to click the banner, and instead of being taken to where the .PNG is stored, it makes more sense to get redirected to fluxer.app
  • Notes for reviewers:None

How to verify

  1. Click the image, verify it takes you to https://fluxer.app/

Tests

  • Added/updated tests (where it makes sense)
  • Unit tests:
  • Integration tests:
  • [x ] Manual verification: clicked the new image and verified redirection works

Checklist

  • [x ] PR targets canary
  • PR title follows Conventional Commits (mostly lowercase)
  • CI is green (or I’m actively addressing failures)
  • CLA signed (the bot will guide you on first PR)

@CLAassistant
Copy link

CLAassistant commented Feb 12, 2026

CLA assistant check
All committers have signed the CLA.

@Jiralite Jiralite changed the title Doc: Update README.md to link banner to website docs: update README.md to link banner to website Feb 12, 2026
@Shadowalker125
Copy link

since you have target=_blank, you should also add

rel="noopener noreferrer"

this protects against reverse tabnabbing

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.

3 participants