theme_bootstrap icon indicating copy to clipboard operation
theme_bootstrap copied to clipboard

MDL-55170 - Fix user enrollment dialog, RTL.

Open gjb2048 opened this issue 9 years ago • 2 comments

Hi Bas,

More of a question to see if the theme needs this patch?

Cheers,

Gareth

gjb2048 avatar Jul 20 '16 16:07 gjb2048

I think theme_bootstrap needs lots of fixes for RTL languages. I am still hoping for an automated solution instead of adding rtl less. So will keep this open for now.

I did track down the less created by Nadav and added it here for later reference.

.dir-rtl {
    &#page-enrol-users {
        .col_userdetails {
            .subfield {
                margin-right: 40px;
                margin-left: 5px;
            }
            .subfield_picture {
                float: right;
                margin-right: 0;
            }
        }
        .enrol_user_buttons {
            float: left;
            .enrolusersbutton {
                margin-left: 0;
                margin-right: 1em;
                div {
                    margin-left: 0;
                }
            }
        }
        #filterform {
            .fitem {
                margin-right: 0;
                margin-left: .3em;
                label {
                    padding-right: 0;
                    padding-left: .3em;
                }
            }
        }
        .user-enroller-panel .uep-search-results .user {
            .count, .picture, .details {
                float:right;
            }
            .options .enrol {
                float:left;
            }
        }
    }
}

bmbrands avatar Jul 21 '16 17:07 bmbrands

Hi Bas,

Yeah, I see. Complicated and time consuming stuff, which is why I removed the auto-flip in Essential for M3.1. Hopefully https://tracker.moodle.org/browse/MDL-55224 will help in the future.

I think the key thing here is feedback from RTL users. Nadav is excellent at testing for this.

G

gjb2048 avatar Jul 21 '16 17:07 gjb2048