-
Notifications
You must be signed in to change notification settings - Fork 19
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
change plugin theme and add UIkit default theme #662
base: master
Are you sure you want to change the base?
Conversation
Wow that is a lot of work. Thanks Ivy. @dengenxp - How should we handle this? I think we should make the denim theme the default theme for Geeklog so I don't mind moving the denim/uikit related files to the default plugin template directory. I don't think we should loose the plugin theme files for modern curve and other similar themes that don't use uikit. We should either keep the original default plugin template files and put them in a modern_curve directory or even keep the original denim plugin template files since they are not based on uikit and put them in the modern_curve directory. What ever we decide on we will have to add a functions.php file to each of the plugin temple directories (see forum for an example) that use uikit so the plugin will tell Geeklog to load uikit if the theme Geeklog is using, is not. I also notice that Ivy has done some changes to plugins/calendar/functions.inc. We should remove that html from the code and put it into a calendar template file. Does that make sense dengen? Do you think we should do it another way? |
ok to make the denim theme the default theme for Geeklog Ben 2016-02-10 21:16 GMT+01:00 Tom [email protected]:
|
Hi, Theme "Default" is based on Denim. and using style.css.php of Modern Curve. Default theme name is Default. @dengenxp, I am happy if you get "Default" partially into "Denim". I changed calendar/functions.php and system/lib-comment.php.
If all theme include UIkit, Geeklog can be change modern cms. |
Tom, It's simple to link UIkit for all theme. Professional is very classical table layout theme. Many user want RWD and HTML5 theme. |
True but not all of our users use a uikit based theme. Plugins now have the ability to include scripts and other files via their own functions.php. I was just saying these need to be added just in case someone uses a theme that doesn't use uikit directly. This way the plugins can include uikit when needed. |
I am hesitant to have the plugin default templates include any classes from uikit. The default theme should be as simple as possible and be allowed to work with the greatest number of themes out there (both old and new). Who knows maybe someone will want to make a Geeklog theme that uses bootstrap. I am sure this person wouldn't really want to have the plugins loading in uikit. That would be a resource hog and would probably create some conflicts. Then again Geeklog has chosen uikit as the framework we will be using going forward. What does everyone else think? Should the default plugin templates use uikit along with the denim plugin templates which I assume we will still be including? |
Bootstrap does not conflict width UIkit. This site use UIkit and Bootstrap, both. UIkit is very light and simple framework. Some CMS include Bootstrap. or Geeklog-UIkit branch. deprecated tag / attribute |
Ivy we are misunderstanding each other. Uikit is the framework we have chosen for Geeklog. Geeklog's Main theme should use uikit. I am not saying we should use anything else. |
Okay.
|
The site http://www.hananokumo.com only displays |
thanks. http://www.hananokumo.com |
@ivywe display is the same
See attachment |
Okay, thanks! |
Please check again. |
Ok it displays nice |
thanks! |
Hi,
All of plugin theme have UIkit style.
UIkit default theme 'Default'.
I checked HTML by using validator.w3.org.
removed table layout and some deprecated HTML
<b>, width="***",