Skip to content

Conversation

@niksirbi
Copy link
Member

@niksirbi niksirbi commented Jul 7, 2025

Description

What is this PR

  • Bug fix
  • Addition of a new feature
  • Other: Docs website

Why is this PR needed?

Updates to the designs of the logo and favicon, following discussions on Zulip.

What does this PR do?

  • The text in the logo is now italicised
  • The favicon now contains a small italicised "m" in its centre
  • The "overview" image (displayed in README and homepage) is updated to incorporate the logo

This is how the website header will show up henceforth:

image

References

Follows up #614
See "#Movement > logo ideas" on Zulip.

How has this PR been tested?

Local docs build.

Is this a breaking change?

No.

Does this PR require an update to the documentation?

It is in itself an update of docs.

Checklist:

  • The code has been tested locally
  • Tests have been added to cover all new functionality
  • The documentation has been updated to reflect any changes
  • The code has been formatted with pre-commit

@codecov
Copy link

codecov bot commented Jul 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (8f38215) to head (919dc50).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #632   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           32        32           
  Lines         1859      1859           
=========================================
  Hits          1859      1859           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@niksirbi niksirbi changed the title Update logo and favicon images Update logo, favicon, and overview images Jul 8, 2025
@niksirbi niksirbi marked this pull request as ready for review July 8, 2025 13:12
@niksirbi niksirbi requested a review from lochhh July 8, 2025 13:13
Copy link
Member

@lochhh lochhh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@lochhh lochhh enabled auto-merge July 8, 2025 13:52
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 8, 2025

@lochhh lochhh added this pull request to the merge queue Jul 8, 2025
Merged via the queue into main with commit 20ca1b9 Jul 8, 2025
27 checks passed
vtushar06 pushed a commit to vtushar06/movement that referenced this pull request Jul 8, 2025
* update logo and favicon images

* updated overview image
@niksirbi niksirbi deleted the logo-italic branch July 24, 2025 08:09
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