Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Use Symfony "dark-mode"-responsive logo in README #53676

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

Merged
merged 1 commit into from
Jan 29, 2024

Conversation

smnandre
Copy link
Member

@smnandre smnandre commented Jan 29, 2024

Q A
Branch? 5.4
Bug fix? no
New feature? no
Deprecations? no
Issues Fix #53105
License MIT

Following #53105 @javiereguiluz uploaded a Symfony "dynamic" logo, featuring a color scheme that adapts to user preferences.

This enhancement allows the README's logo to appear sharper and cleaner in dark mode (on Github.com, in PHPStorm, ...)

<svg>
    <style>
+        @media (prefers-color-scheme: dark) {
+            .symfony-logo-circle, .symfony-logo-letter { fill: #fff; }
+            .symfony-logo-circle-letters { fill: #000; }
        }
    </style>
    <!-- SVG code -->
</svg>
Before After
before_dark after_dark

( Thank you again @javiereguiluz )

@carsonbot carsonbot added Status: Needs Review DX DX = Developer eXperience (anything that improves the experience of using Symfony) labels Jan 29, 2024
@carsonbot carsonbot added this to the 7.1 milestone Jan 29, 2024
@carsonbot carsonbot changed the title [DX] Use Symfony "dark-mode"-responsive logo in README Use Symfony "dark-mode"-responsive logo in README Jan 29, 2024
@nicolas-grekas nicolas-grekas modified the milestones: 7.1, 5.4 Jan 29, 2024
Following symfony#53105 @javiereguiluz uploaded a Symfony "dynamic" logo, which color scheme depends on user preferences.

Thus allowing the README's logo to look sharper and cleaner in dark mode (on Github.com, on PHPStorm, ...)

```diff
<svg>
    <style>
        .symfony-logo-circle, .symfony-logo-letter { fill: #000; }
        .symfony-logo-circle-letters { fill: #fff; }
+        @media (prefers-color-scheme: dark) {
+            .symfony-logo-circle, .symfony-logo-letter { fill: #fff; }
+            .symfony-logo-circle-letters { fill: #000; }
        }
    </style>
    <!-- SVG code -->
</svg>
```
@nicolas-grekas nicolas-grekas changed the base branch from 7.1 to 5.4 January 29, 2024 20:40
@nicolas-grekas nicolas-grekas force-pushed the dx/readme-logo-dark-mode branch from cbb755d to eae4ec9 Compare January 29, 2024 20:40
@nicolas-grekas
Copy link
Member

Thank you @smnandre.

@nicolas-grekas nicolas-grekas merged commit 8c0ac47 into symfony:5.4 Jan 29, 2024
@antonkomarev
Copy link

Screenshot_2024-01-30-08-24-32-911_com.github.android.jpg

In the official mobile app. Android with dark mode interface.

@smnandre
Copy link
Member Author

@antonkomarev could you try on the 5.4 branch just to be sure ?

@derrabus
Copy link
Member

derrabus commented Jan 30, 2024

On my iPhone:

GitHub app

image

Safari

image

Both screenshots show the 5.4 branch, but it's the same on 7.1.

weaverryan added a commit to symfony/ux that referenced this pull request Jan 30, 2024
This PR was merged into the 2.x branch.

Discussion
----------

[DX] Use dark-mode Symfony logo

See symfony/symfony#53676

| Before | After |
| - | - |
| ![before_dark](https://github.com/symfony/symfony/assets/1359581/a01633d4-c0e5-45fd-b158-8f007bd22144) | ![after_dark](https://github.com/symfony/symfony/assets/1359581/ebacc787-b049-435e-9615-eead19743f39) |

Commits
-------

7260c6b [DX] Use dark-mode Symfony logo
@smnandre
Copy link
Member Author

Both screenshots show the 5.4 branch, but it's the same on 7.1.

Just to be complete, could you show how it renders in light mode ?

@derrabus
Copy link
Member

Certainly!

image

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DX DX = Developer eXperience (anything that improves the experience of using Symfony) Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DX] Symfony logo in dark mode
5 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.