Skip to content

[Site] Remove light mode & clean templates #1822

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

Closed
wants to merge 3 commits into from

Conversation

smnandre
Copy link
Member

@smnandre smnandre commented May 3, 2024

Q A
Bug fix? no
New feature? no
Issues Fix #.
License MIT
  • Dark mode for everyone
  • Restructure template organisation
  • Rework banner
  • A lot of tiny things

@carsonbot carsonbot added Site ux.symfony.com Status: Needs Review Needs to be reviewed labels May 3, 2024
@smnandre smnandre requested a review from kbond May 3, 2024 05:11
@javiereguiluz
Copy link
Member

I'd like to ask you to reconsider the removal of light mode. For folks living in countries with lots of Sun and for folks working in well-lit workplaces, light mode is a must 🙏

But, if maintaining an additional theme takes a lot of effort, then go ahead and remove it (better spend that effort on other more useful things). Thanks!

@smnandre
Copy link
Member Author

smnandre commented May 3, 2024

Thank you for your comment @javiereguiluz

As you mentioned, this is primarily a matter of resources.

Currently, I have three demos on my task list, approximately fifty new assets, and four substantial components. Over the next few weeks and months, we plan to create many new pages and components, accompanied by numerous design tasks.

For a limited time, I'd like to focus solely on one theme to gain the agility I currently lack.

My original "plan":

This week:

  • Remove the light mode (this PR)
  • Start removing unused CSS

In the following weeks:

  • Significantly reduce our use of Bootstrap
  • Strengthen our HTML/CSS codebase
  • Implement all the new features we have planned

Subsequently:

  • Reintroduce the light theme.
  • Add options for dimmed, contrast settings, etc.

I realize this approach may not be ideal for everyone, and I apologize in advance if we decide to proceed.

I'm not making any promises yet, but I will take some time to consider alternative plans.

Let’s pause for 24 hours before making any decisions.

@javiereguiluz
Copy link
Member

Simon, it sounds like a good plan because removing the light mode would be temporary and it will be hopefully readded in the future 👍

Just another comment: if you want to go faster and be more productive, I think you should use Bootstrap components more instead of less. Modern Bootstrap allows to customize almost everything using CSS properties, so you can use Bootstrap without looking like default Bootstrap.

@Kocal
Copy link
Member

Kocal commented May 4, 2024

Hey, I was like 😱 when reading the PR's title, but I'm more reassured after reading the whole PR. Yeah, please keep the light theme 🥲 🙏🏻

People are too "hyped" by the dark theme, but the truth is that white text on dark background is harder to read than black text on light background.

More generally, you may want to use a light theme during the day, and a dark theme during the night, this way you don't have to increase a lot your monitor's brightness (which damages your eyes).

Thanks 🙏🏻

Copy link
Contributor

@WebMamba WebMamba left a comment

Choose a reason for hiding this comment

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

Hey I was reviewing this PR, and I end up with a question: The problem we have here is it really because it's hard to maintain two theme or because the theming system is not implemented well? If we struggle to have two theme because the code is not good enough we should refactor this code! In front development theming is a huge topics, and since SymfonyUX is made also for front end development we should show the exemple and demonstrate that is possible to have a correct design system with SymfonyUX.

I think we should take a little step back here, look over and find the reason that made this theming task over complicated. Make a list, and try to figure out if some issue can't be solve.
For example when digging into the scss file I see that a lot of colors and value are hard coded. Maybe we can start by that?

WDYT ?

@smnandre
Copy link
Member Author

smnandre commented May 5, 2024

it's hard to maintain two theme

Never said that... i said it's hard "right now", so we agree on this. It should be easy and i'm confident it will be once we're done here.

For example when digging into the scss file I see that a lot of colors and value are hard coded.

I know, and even if i added some lately, i can garantee you there are less than when i started working on the website 6-8 monthes ago. I'm sure you're not pointing at anynone saying that, but we need to have some pragmatic approach here.

The website is open-source, but it's not a project like the "Symfony Demo", and its PHP code does not follow all best practices neither, because we focused on delivering content to showcase the SymfonyUX packages, and i think on this point there have been massive improvments in the past monthes.

we should refactor this code

Did you read my comments or am i that bad expressing myself (please don't answer haha 😆 )

That's entirely the concept of this PR, which will lead to other ones as I said.

We need to move quickyl, as we have many parrallel projects / content incoming.

I could have said: let's have a long and "V-shape" project, and when we're ready we will switch all the front at once, but i'm convinced it's not the good idea here...

So the idea here is to lighten temporarely the front-end tools we use, as Bootstrap has a very specific way of doing things, to rebuild on saner grounds.


I bounce on what @javiereguiluz said about resources, i'm personnaly 100% convinced our energy should be spent on 3 things

  • features: creating more component and/or adding new cool stuff on the existing ones, and fixing all the little bugs/unexpected/magical behaviours
  • DX: easing the usage of Symfony UX for the everyday life of developers. Meaning better errors handling, debug tools, makers, documentation (which tool / method for which need, etc..)
  • spreading the word: as we discussed many time together, and as we saw at Symfony Live Paris, a lot of people do not know what is SymfonyUX, what the big components allow, and we need to spread the word. For this the cookbook is a brilliant idea / thing, i repeat once more a blog / newsfeed is required too, as is rebranding the UX communication materials ("baseline" / "punchlines" / even github tags...) to make thinks a lot more in harmony with the current state of the project

So, the website should be a tool dedicated to those goals, and that should make us win time in fullfilling them, not somethings that make us doubt or think twice.

Today, it is not that tool, and to became this tool, we need major changes in the way it's coded (in HTML/Twig mainly, CSS beeing a consequence of it).

So what i'm pushing here is to do those changes "live", with the minor drawback to temporarely disable features (knowing that most of them did not exist only a couple monthes before)

The final goal is the same as it always was: a website to promote and showcase "Symfony UX" components in the best way we can.

@smnandre smnandre mentioned this pull request May 6, 2024
@smnandre
Copy link
Member Author

smnandre commented May 6, 2024

PR replaced by #1827 - so i'm closing this one.

I'll try to refacto the front + implement all the new features while keeping the light mode.

But i'm not making any promise here.

@smnandre smnandre closed this May 6, 2024
kbond added a commit that referenced this pull request May 8, 2024
This PR was squashed before being merged into the 2.x branch.

Discussion
----------

[Site] Clean templates

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| Issues        | Fix #
| License       | MIT

Replaces #1822

All the clean / fixes / CS   (without removing the light mode for now)

Commits
-------

3de425e [Site] Clean templates
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Site ux.symfony.com Status: Needs Review Needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants