Tabler UI#2892
Conversation
Merged from mariofix/flask-admin-tablerui Tabler version 1.4.0
| theme_base: str = "gray" # "gray" | "neutral" | "slate" | "zinc" | "stone" | ||
| theme_font: str = "sans-serif" # "sans-serif" | "serif" | "monospace" | "comic" | ||
| theme_radius: str = "1" # "0" | "0.5" | "1" | "1.5" | "2" | ||
| theme_use_cdn: bool = True # From where to load tabler files |
|
This is huge! Great work, really appreciate the effort, thanks! |
|
@ElLorans What are the steps that we need to do to push this to the finish line? |
|
Replying to my 2 comments and fixing the tests (but this I can do myself) |
| f'<span class="fa {fa} glyphicon glyphicon-{glyph} ' | ||
| f'<span class="fa {fa} glyphicon glyphicon-{glyph} ti ti-{ti} {color} ' |
There was a problem hiding this comment.
This feels like a breaking change. What's the impact on bootstrap4 template?
There was a problem hiding this comment.
Ignored if tabler icons are not loaded, same for glyphs, deprecated since long ago but the class is still there
| """ | ||
| Tabler 1.4.0 theme for Flask-Admin. | ||
|
|
||
| Ships Tabler UI assets locally (CSS, JS, icon webfonts) so no CDN is |
There was a problem hiding this comment.
this refers to shipping locally but that will need a separate PR?
There was a problem hiding this comment.
Up to @mariofix, I'm fine with doing only CDN for the first version.
There was a problem hiding this comment.
Yea, you said first pr with cdn to.make it simpler, there are like 5 or 6 static files
@ElLorans Fixed the tests in mariofix#1 cc @mariofix |
3 participants
This PR adds Tabler UI into Flask-Admin, an early version.
fixes #2443
See
tablerfolder included in examplesoptions allowed
you can go to https://preview.tabler.io to see combinations
let me know what you think