View Issue Details

Jump to NotesJump to History
This bug affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
10955Feature requestsTheme editorpublic2016-04-15 09:212016-04-15 12:50
ReporterDenisChenu Assigned Toollehar  
PrioritynoneSeverityfeature 
Status assignedResolutionopen 
Summary10955: Remove the usage of "radio" class from BootStrap
Description

BootStrap use .radio for the radio wrapper.

This class does 2 think
.checkbox label, .radio label {
cursor: pointer;
font-weight: 400;
}

and
.checkbox input[type="checkbox"], .checkbox-inline input[type="checkbox"], .radio input[type="radio"], .radio-inline input[type="radio"] {
margin-left: -20px;
position: absolute;
}

But this 2 think is not needed, the ansolute position is something good, but must set to top:50% : we have morte text than athe little form in BootTrap

Additional Information

In fact : BootStrap is done to do website, not from website, the BootStrap forms are for "little" form : we have big form with a lot of element.

And here : the radio absolute in td is really a bad idea .

I know i can use CSS to update it : but bootstrap.css coem from template, not from "core" : if it come from core : decision was made to don't use Casacading a lot.

TagsNo tags attached.
Bug heat6
Story point estimate
Users affected %

Users monitoring this issue

There are no users monitoring this issue.

Activities

ollehar

ollehar

2016-04-15 10:41

administrator   ~37426

This is the styling we use: http://flatlogic.github.io/awesome-bootstrap-checkbox/demo/

Not sure what your point is... Do you have a display error?
DenisChenu

DenisChenu

2016-04-15 10:47

developer   ~37429

You think nonly of "default" and "included" template.
But template can be done by any other people.

http://testing.sondages.pro/masterall/index.php/49728?lang=fr

Then We need to remove Cascading from Bootstrap.
LouisGac

LouisGac

2016-04-15 10:56

developer   ~37431

in fact the bug come from form.css applying a display: block; to the tds
LouisGac

LouisGac

2016-04-15 10:57

developer   ~37432

and you can disable bootstrap in a custom template from the config.xml
DenisChenu

DenisChenu

2016-04-15 11:49

developer   ~37438

If you disable bootstrap for a template : you broke slider ....
ollehar

ollehar

2016-04-15 11:54

administrator   ~37440

So remove "display: block;" from the radio wrapper class?
DenisChenu

DenisChenu

2016-04-15 12:19

developer   ~37444

The point is that :

  • BootStrap have some great system
  • BootStrap whole have some bad feature

Actually : forms of bootstrap are not really good for big forms ... we can "updfate/fix" due to casacding.

But if we use cascading : no need to add bootstrap.css in each template :; add it everywhere with ALL default from bootstrap (color too).

A,nd use bootstrap-theme.css in template .....
LouisGac

LouisGac

2016-04-15 12:48

developer   ~37447

slider js is in template side, so you can add your own slider js if you want
DenisChenu

DenisChenu

2016-04-15 12:50

developer   ~37448

You mean : we fix.add feature slider.js in core : must fix/add feature in all included template ....

Strange for me ....

Issue History

Date Modified Username Field Change
2016-04-15 09:21 DenisChenu New Issue
2016-04-15 09:21 DenisChenu Status new => assigned
2016-04-15 09:21 DenisChenu Assigned To => ollehar
2016-04-15 10:41 ollehar Note Added: 37426
2016-04-15 10:47 DenisChenu Note Added: 37429
2016-04-15 10:56 LouisGac Note Added: 37431
2016-04-15 10:57 LouisGac Note Added: 37432
2016-04-15 11:49 DenisChenu Note Added: 37438
2016-04-15 11:54 ollehar Note Added: 37440
2016-04-15 12:19 DenisChenu Note Added: 37444
2016-04-15 12:48 LouisGac Note Added: 37447
2016-04-15 12:50 DenisChenu Note Added: 37448