search

okkur / syna

Highly customizable open source theme for Hugo based static websites
https://syna.okkur.org/demo/
Apache License 2.0
250 stars 133 forks source link

wip: Add variant capability #812

Open mpourismaiel opened 4 years ago

mpourismaiel commented 4 years ago

What this PR does / why we need it:

Special notes for your reviewer:

Release note:

stp-ip commented 4 years ago

Interesting solution to trim the prefix. That way we can migrate to bootstrap/ without impact or at least none that I can think of. Kinda like that solution, but this needs a comment or at the very least a commit message explanation on why this was done.

stp-ip commented 4 years ago

Also not sure, why the modal close "x" is being shown in firefox bottom right corner on the tailwind dev page: Screenshot_2020-08-05 Tailwind page

Also fascinating that it does look similar to bootstrap :)

mpourismaiel commented 4 years ago

This was just to try and implement a simpler fragment using the same design but using Tailwind. I have to figure out a way to simplify the "install" process. If the user doesn't have to do anything besides loading the fragment and configuring it, I'm way happier. If it's possible, move the tailwind config and try and figure out a way to handle postcss in another way. But still this PR is to show how a fragment using tailwind will be developed and used.

Theme of the fragments need to be different IMO. Or we can implement the same designs and allowing the user to choose between them, kind of a basis for custom fragments, but I don't like that idea. I prefer a totally new theme. Tailwind UI would be awesome. Or I can go ahead and design a new theme.

Colors have to be fetched from config.toml. Colors would be the same as Bootstrap by default, but IMO we can add recommended color schemes to docs, maybe even for Bootstrap.

I'll continue working on Tailwind and "variants" if you agree with how the structure looks and how it's handled (except for colors not imported and postcss/tailwind config files), but let me know if you think a new theme should be developed or not.

stp-ip commented 4 years ago

Generally agree with the direction. Question is what do you define as a new "theme"?

mpourismaiel commented 4 years ago

A new style in general for the Tailwind variant, if we decide not to go with Tailwind UI or add Tailwind UI as another variant. Tailwind can be used to implement any style, Bootstrap style as well. We can implement the entire Bootstrap look using Tailwind or we can take this opportunity to create a new style.

stp-ip commented 4 years ago

We could think of a new theme, but we also have to keep a check on the time spend as it's mainly duplicating functionality. I kinda like the thought of having a tailwindUI theme with all the things it enables or doesn't.

stp-ip commented 4 years ago

Needs a rebase due to the dep updates, but you wanted to finalize the general design ideas before merge anyway or?

mpourismaiel commented 4 years ago

I'm gonna extract the variant code and create a pr for that and will work on design and create a few fragment using that. So this pr will be a point of reference and shouldn't be merged. Is that a good plan?

stp-ip commented 4 years ago

Sounds like a plan :)

On Fri, Aug 7, 2020, 2:32 PM Mahdi Pourismaiel notifications@github.com wrote:

I'm gonna extract the variant code and create a pr for that and will work on design and create a few fragment using that. So this pr will be a point of reference and shouldn't be merged. Is that a good plan?

— You are receiving this because your review was requested. Reply to this email directly, view it on GitHub https://github.com/okkur/syna/pull/812#issuecomment-670493486, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA4YAXAJ4ZUI4ERZNJDJP2DR7PX43ANCNFSM4PHQSEUA .