bootstrap-markdown icon indicating copy to clipboard operation
bootstrap-markdown copied to clipboard

Bootstrap 4 support

Open mattez opened this issue 9 years ago • 14 comments

Bootstrap 4 support?

mattez avatar Jan 08 '16 12:01 mattez

I guess that is something the library should support, but bootstrap 4 is currently in alpha. We can start on it when it reaches beta as from then on there will be no api changes!

acrobat avatar Jan 08 '16 12:01 acrobat

Great. Thanx for reply. (Anyway: I`m going to try it...if it works without changes.)

mattez avatar Jan 08 '16 12:01 mattez

I think the classnames will change a bit, but please try it and give us some feedback! You can also start a PR to add BS4 support

acrobat avatar Jan 08 '16 12:01 acrobat

:+1:

toopay avatar Jan 08 '16 18:01 toopay

First observation: Bootstrap 4 dropped Glyphicon support. So there no icons but buttons works fine:

screenshot from 2016-01-11 15 00 54

mattez avatar Jan 11 '16 14:01 mattez

Second observation: Bootstrap has dropped the hidden class due to incompatibility with jquery's show(), hide() methods.

blackfyre avatar Apr 27 '16 14:04 blackfyre

To hide an element, you need to use attribute hidden. http://v4-alpha.getbootstrap.com/content/reboot/#html5-hidden-attribute

ex: <button class="btn-sm btn btn-primary" hidden>button</button>

Magicalex avatar Oct 18 '16 19:10 Magicalex

i haven't really had any issues with bootstrap4. the only thing i had to do was add iconlibrary: 'fa' as a parameter

itsmill3rtime avatar Feb 23 '17 01:02 itsmill3rtime

Thanks guys. I'll try to conduct wider test this week.

toopay avatar Feb 23 '17 13:02 toopay

Hi! @toopay is there any news? I would like to use this project but we are using Bootstrap4 since it's stable :)

LucileDT avatar Jan 25 '18 09:01 LucileDT

Note that in order to play well with fontawesome5 if you do treeshaking one needs to edit bootstrap-mardown.js and replace line 1003 fa-header by fa-heading, line 1070 fa-picture-o by fa-image.

Maybe we can have a 'fa-5' option to deal with this situation?

As for bootstrap4 everything looks fine. As @itsmill3rtime said, one needs to explicitly call the plugin on the element to add the iconlibrary: 'fa' option.

Cheers :)

NicolasCARPi avatar Mar 30 '18 12:03 NicolasCARPi

the way this is done normally is we bump a major version to the plugin, create a branch for the current BS 3 support (2.x) and then put BS4 support in master (3.x)

lodev09 avatar Mar 30 '18 12:03 lodev09

The default glyphicon icon doesn't load. Will try FontAwesome.

niknokseyer avatar Apr 03 '18 03:04 niknokseyer

Note that in order to play well with fontawesome5 if you do treeshaking one needs to edit bootstrap-mardown.js and replace line 1003 fa-header by fa-heading, line 1070 fa-picture-o by fa-image.

You can fix the missing icons using fontawesome5 by adding this two lines to your css, so you do not need to alter the javascript:

.fa-header::before { content: "\f1dc"; }
.fa-picture-o::before { content:"\f03e"; }

hangya avatar Oct 04 '19 08:10 hangya