-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Change center alignment of Joomla! logo on issis login page #4275
Conversation
oops, I see the commits of my forked repository are attached to this PR. Is that a problem and how can I prevent this in the future? |
I'm using the Dutch language from here: http://joomlacode.org/gf/download/frsrelease/19702/160147/nl-NL_joomla_lang_full_3.3.3v1.zip And i can confirm the issue an the patch (also for the german language pack) Only as note for testers: Clean your cache and/or retry with a alternative Browser if it don't work "out of the box" :D So @test successful thanks @n9iels |
i don't think this is a problem here. But how you do this change? via Github UI? If yes you can fork for every issue/PR from the main repo (accessing the file on https://github.com/joomla/joomla-cms and try the edit button) ;) If you use any tool for git this function calls IIRC anything like "squash the commits". e.g. something like: |
Yes, I forked the repo, maked my changes and create a pull request. With the edit function you can't edit multiple files? Or you can? |
yes you can.
Problem: Every change will be a new commit. 😄 |
oke, thanks for the explanation :-) |
As long as the resulting PR only contains the changes you intended to do, it's absolutely fine 😄 |
Have you tested what it gives for RTL languages? |
I didn't test that, what result you expected? I think it works fine for that languages to. The |
Tested: It looks like you have to add the class in template_rtl. less and use generatecss to get it in template_rtl.css with these changes
|
oops, forget to edit the tempalte_rtl.css and template_rtl.less. But I have no idea how to add that changes to this PR. Can someone explain please? @infograf768 I think the changes must be made in the .less files, than they can build to .css files. I don't know how to do that on gihub... |
We usually use cli generatecss.php (for the rtl template as well as the LTR one) (also look better at the css I posted above: I changed left to right |
oke that sounds great. but still have no idea how to do that.... |
oke after a couple of hours trying and frustrating I run the generatecss.php (I'am sorry for the many changes, something went wrong a couple of times). |
Preparing new PR to replace this one. |
Please test |
I am closing this in favour of #4278 |
Set to "closed" on behalf of @zero-24 by The JTracker Application at issues.joomla.org |
In some languages the alignment of the Joomla! logo on the issis login page isn't in the middle of the page. This is because the translation of Go to site homepage. has more characters in some languages (in dutch for example the translation is: Ga naar de startpagina van de website).
This css and less changes align the logo in the middle of the page regardless the language