-
-
Notifications
You must be signed in to change notification settings - Fork 364
[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
Conversation
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! |
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:
In the following weeks:
Subsequently:
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. |
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. |
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 🙏🏻 |
There was a problem hiding this 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 ?
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.
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.
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
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. |
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. |
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
Uh oh!
There was an error while loading. Please reload this page.