-
Notifications
You must be signed in to change notification settings - Fork 98
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
New default theme 'Bootstrap 4', Part 2 #672
Comments
Any ideas for the UI for that? |
Minimalistic UI such as for file attachment to email. 3 buttons: select local file name, export, import EDIT EDIT2 EDIT3 |
Anything that can upload executable code (like config.php) is a no-go. But using JSON would be an option. |
I j ust had a look at the momentary master. The "offcanvas nav" is 100 % broad - and so the seek-mask is too broad and also the right side. It is full open at first level and overshadows the content too much. In the bootstrap 4.5 examples is an offcanvas , at which i can set .offcanvas-collapse.open to width: 50% to solve that. Here it is different (media width and header class="main-header ). (( Though i still prefer https://codepen.io/Mineslu/pen/MzGagm or https://codepen.io/vmitsaras/pen/gwGwJE or https://codepen.io/vmitsaras/pen/pWNrEy and https://github.com/vmitsaras/js-offcanvas - which is the probably future)). |
Hm, but I haven't yet pushed anything from the alternative mobile menu styles, but it's in the works. So all we currently have in the master is the default pulldown nav. Yes, that's basically what will be selectable. |
This variant will be called 'Slide Over' in our theme. 'Off Canvas' will be similar but 'push' the rest of the page off the viewport. |
I see i was in too much of a hurry. In my (coming) BT5biz - Template the offcanvas destroys the sticky nav. There must be some incompatible code in the css(needs restyling and evtl some js-code) . Thats why i prefer the js-offcanvas, which needs jquery (or i wait and have a look at Your solution...). |
Sounds like you have to take care not to load any js or css assets unrelated to the current theme/layout.
Typesetter will continue to use jQuery for an indefinite period – so it is there even with your Bootstrap 5 theme. |
Can be adjusted in variables.scss or via the Scss tab in Layout Editor. Find and change… $dropdown-padding-y: 0.5rem;
$dropdown-item-padding-y: 0.25rem;
$dropdown-item-padding-x: 1.5rem; |
FYI: as noted earlier position:sticky does not work in overflow:hidden context. Most off-canvas solutions apply overflow:hidden to the body element. That's the main reason why I'm still working on the different menu styles – quite tricky to get them to work with all possible settings. But I'm almost done. |
Here come the 'Hamburger Menu Styles' with bba14f4 First, there are a few other changes too…
For the mobile menu styles: They certainly still need some tweaks. I haven't yet tested all relevant customizer combinations. I'll probably also add a thumb-friendly 'Slide Up' style, which is currently trending and makes sense. The animated hamburger is a bit playful and maybe not to everyone's taste. Although there is already a Scss variable, I haven't added it to customizer yet. Shall we add it? Tell me what you think. |
mobile menu styles and a few other improvements. I will comment on that…
Important! I forgot to mention that 'Off Canvas' and 'Slide Over' menus can be swiped away, but that requires jquery-touch which is only available since this commit You will see JS errors and non-working menus without it! |
It seems only to work for footer. Could you check it, please. P.S. |
Slide Over appears and hides on window resize, e.g. on screen rotation. UPD |
Yep, fix is here 5b76151 |
Yep, that definitely still needs some tweaking. EDIT: LOL, that's why I started the JS breakpoint detection stuff. Once it worked I couldn't remember what I made it for :) |
I should trust the DONT-YOU-WRITE-INLINE-CSS bell ringing. What a whole lotta mess. This is JS event fireworks! m( Seems like I fixed it EDIT: Beware, small artifacts ahead. When the navbar expands in open-menu-state, original Bootstrap transitions chime back in and cause some naughty little blinking. But nothin' serious :) |
Great - But maybe i overlooked it : Is the position of the main menu fix attached to the searchfield und not (yet) adjustable in the middle ? (like d-flex justify-content-between ) The searchfunction may look a bit more stylish as in https://codepen.io/gungorbudak/pen/ooKNpz (1st line) and is rather high... The Login and sitemap may have an icon instead . |
No. At least not yet. Currently we need to change this line in template.php
Lots of things could be more stylish, but it depends on what different people think is stylish. |
It's really easy to implement - so yeah, let's do it. |
Done with 3021cc6 |
I ended up with this d1be675 But we can still discuss everything. While I like the current dark header colors, fresher, friendlier colors might make better default settings. I would love to see some great examples from you guys! |
I dare to propose one more search setting "Size". First option - big, the other - compact. Because for mobile first sites, BS's default large fields fit better. |
Well - if You look for a big navbar - You can try https://bootsnipp.com/snippets/a6aV0 |
I assume they simply take CIELab L50 (psychometric lightness) as a threshold – which is not all bad. If so, they just barely passed their own test :) |
Can't confirm. Scss compilation must have gone wrong. Please clear CMS chache and try again UPD: Ok, it's the starter layout.... I check it |
Fixed with 5ff7aa1 |
Yes, I'm afraid that we also need dropdown colors customizable. |
I can understand the work for You that is behind it , but i thinkcustomizer is quite comfortable for users. In this case the menucolors and the dropdown-colors may differ in opacity 0.8 or similar. Only some bugs may need a quick fix and this is a RC - so never mind (...or take refuge...). I hope the 5.2 is coming soon. P.S. The sumenu-li's are blue and not like the color of the toplevel - lis's. (dropdown colors customizable. is not really necessary) |
In my opinion, the power of this template is not in its full customization, but in a balanced list of options allowing beginners to get dozens of cool and almost unique designs. |
The SCSS - problem may be solved, but the problem that the submenu has a blue li while the top li is black(or another color) is still unsolved. Something like |
@gtbu take it easy, several things are not yet finished |
It's meant to be thin or better call it 'light'.
Well, we do not actually have a real logo or 'trademark'. Nobody would pay the protection fees. |
In fact, not yet tested. |
Looks OK here on Opera Default size fixed with 158f4bb |
I have 125% scale in Win7. Maybe it somehow affects svg rendering. I don't know. Anyway, with the last fix, the logo looks better to me. |
Dropdown menu customization added with 9348998 With that amount of color settings, it should be possible to define – let's say 6 – basic colors and show them as swatches next to every color picker field. |
That's so that the content doesn't scroll over the headers. I changed it to use the Bootstrap variables. That's what we have them for ;) |
Never was. The actual dropdown only starts next to the checkbox. |
Yep. My bad |
Yes, this helps. |
I can't catch the moment, but sometimes some changes in layout cause the complementary header to change its state from show: never to show: always. Have you ever faced this issue? UPD |
Should be fixed with this PR #682. |
Continuing this issue in a new thread
The text was updated successfully, but these errors were encountered: