Skip to content
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

MDL-55170 - Fix user enrollment dialog, RTL. #437

Open
gjb2048 opened this issue Jul 20, 2016 · 2 comments
Open

MDL-55170 - Fix user enrollment dialog, RTL. #437

gjb2048 opened this issue Jul 20, 2016 · 2 comments

Comments

@gjb2048
Copy link
Collaborator

gjb2048 commented Jul 20, 2016

Hi Bas,

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

Cheers,

Gareth

@bmbrands
Copy link
Owner

bmbrands commented Jul 21, 2016

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;
            }
        }
    }
}

@gjb2048
Copy link
Collaborator Author

gjb2048 commented Jul 21, 2016

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants