alanaktion / proboot Goto Github PK

Tabler overrides .nav-item and .nav-link without enough specificity, causing .navbar-nav to display incorrectly on mobile.

In particular, the .close buttons still use hardcoded Feather Icons when they shouldn't have any dependencies. Not that I don't love Feather Icons <3

A modified version of the tagsinput.js lib could work well with Tabler's tags

This is fairly easy, we could add something like this, which introduces a new 4:3 class for the main imagepicker element:

.imagecheck-4-3 {
    width: 100%;
    .imagecheck-figure {
        padding-bottom: 75%;
    }
    .imagecheck-image {
        position: absolute;
        object-fit: cover;
        width: 100%;
        height: 100%;
    }
}

Bootstrap 4.2 includes a new switch component. For improved compatibility, it would be good to update our switch to just style their component instead of including our own.

Right now the checked state only adds a small checkmark to the top-left of the image. We should make that check more prominent as well as add a strong outline to make it easier to distinguish a selected image from the rest.

Using background-image in an inline style attribute can have security implications, and won't play well with CSP. We should support using an <img> tag as an alternative in the avatar component. This should also support the avatar tags.

The hard-coded body height has several unintended side effects, including breaking the behavior of position: sticky elements. Any components that rely on it should be updated to not require this weird style.

https://blog.getbootstrap.com/2019/02/11/bootstrap-4-3-0/

This may just work with no changes, but we should verify everything still works.