-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Bootstrap 4 support #1159
Comments
++1 |
同问。 |
+1 |
++1 |
+1 but I don't think it's going to happen. The status in the README is not-maintained and last commit over a year ago. Im not a javascript/frontend wiz but I would hope the maintainer would consider releasing this to someone that wants to take it over because it is pretty nice 😄 |
@haggy This is an open-source project and hosted on github. Anybody can fork it and make a modification. What do you mean by "release this"? |
@getvivekv Yes anyone can fork it but it's better if the maintainer will officially state that they are no longer maintaining this project. That way the project can be handed over to someone else or the "main" fork can officially say they are the active version of the project. It's a matter of semantics but it helps keep the landscape cleaner. |
It's also made more important by the fact that the Bootstrap 4 site still links directly to this repo. |
@haggy The developer clearly said this is not longer being maintained on their home page. Please take a look at it |
@getvivekv Hence my original comment:
That means making someone else the official Maintainer in GH so that devs can submit PR's again. Right now there are 40 open PR's. |
+1 |
BS4 Inline: $('.selector').editable()
.on('shown', function (e, editable) {
const input = editable.input.$tpl.attr('class', 'form-control form-control-sm');
const container = input.parent().parent();
container.find('.btn-default').removeClass('btn-default').addClass('btn-secondary');
container.find('i.glyphicon-ok').removeClass('glyphicon glyphicon-ok').addClass('material-icons').text('done');
container.find('i.glyphicon-remove').removeClass('glyphicon glyphicon-remove').addClass('material-icons').text('clear');
container.find('.editable-clear-x').addClass('material-icons').css('font-size', '14px').text('backspace');
}) |
For those of you who are struggling to implement it on Bootstrap 4, maybe this can help. For Bootstrap 4 its work, check this #1189 (comment) |
Hi
Do you plan to introduce support for Bootstrap 4?
The text was updated successfully, but these errors were encountered: