Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heylookalive’s picture

Can someone explain the overrides that are currently in place? I've found a few bits which are off but I don't want to wade in and disrupt legacy things without knowing why they're there first. Also where's the less file?

  • Checkbox & radio button label line height
  • Autocomplete textfield spinner position
  • Password strength meter position
  • Negative margin on .help-block, .control-group .help-inline isn't needed

Could also improve the autocomplete drop down to look a bit nicer too.

Denes.Szabo’s picture

Status: Active » Needs review
FileSize
1.76 KB

I just attached my version. I started to work on the bootstrap3 subtheme, my lessc just found a few missing variables. I replaced them according to the variables.less. Maybe, I did not clearly understand the bootstrap theme (naming conventions, etc), but I hope, my little work helps.

markhalliwell’s picture

Status: Needs review » Needs work

@Denes.Szabo thanks for the patch! This certainly helps get us on the right path, but still a lot of work to do on this issue :)

Denes.Szabo’s picture

Yeah, I agree…

Have you any vision/plan, what, how? I have a little time now, so I can work on it.

markhalliwell’s picture

In reality, this issue should be the last one to patch (IMHO). We won't necessarily know what other issues could affect this one, until we're close to an RC.

markhalliwell’s picture

markhalliwell’s picture

Status: Needs work » Needs review
FileSize
11.39 KB
12.91 KB

Here is the rest of what I can tell needs to be updated (so far). Also compiled it.

kslonka’s picture

+++ b/bootstrap_subtheme/less/overrides.less
@@ -173,12 +162,11 @@ input, textarea, select, .uneditable-input {
   height: auto;

height: auto; causes firefox to render this incorrectly. Do we really need that?

markhalliwell’s picture

Status: Needs review » Reviewed & tested by the community

Probably not, no. They've changed their styling, obviously, so I'll remove it. We can always put it back in/modify it if needed. I'll add this fix after patching.

markhalliwell’s picture

Status: Reviewed & tested by the community » Fixed

Committed #7:7660918 to 7.x-3.x.
Committed #8:f5b7f0f to 7.x-3.x.

markhalliwell’s picture

Version: 7.x-3.x-dev » 7.x-3.0-beta1
kslonka’s picture

We should remove @media @normal, @wide from body styling because we use navbar that is fixed on top as default. (it should always have padding-top)

@media @normal, @wide {
    padding-top: 60px;
  }
markhalliwell’s picture

@kslonka, open new issues please.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.