User tests: Successful: Unsuccessful:
Pull Request for Issue #9822
Status | New | ⇒ | Pending |
Labels |
Added:
?
|
Category | ⇒ | Templates (admin) |
Title |
|
Just checking the code i think this will add a padding-top: 12px;
to ALL the form that uses the .form-horizontal .control-group
css class...
Yes this is not the correct solution at all
On 15 April 2016 at 12:49, andrepereiradasilva notifications@github.com
wrote:
Just checking the code i think this will add a padding-top: 12px; to ALL
the form with that uses the .form-horizontal .control-group css class...—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#9930 (comment)
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
@brianteeman working on it !
So ,should I create a new class for this particular form so that the other forms don't get affected ?
i think the problem you're referring only applies to tinymce modals.
And this is not the way to solve that problem because it would have side effects.
IMHO you should close this and take time to think of a way of solving the problem without the side effects.
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-04-16 20:32:55 |
Closed_By | ⇒ | rdeutz |
Strange, isn't it? Bug in Github?
It is because both PRs include this commit: GABBAR1947@890f048
And @rdeutz merged the other PR.
Please do NOT edit the css file directly.
You must edit the less file and then run the generatess.php to update the css