Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B bootstrap
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 263
    • Issues 263
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 114
    • Merge requests 114
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Bootstrap
  • bootstrap
  • Merge requests
  • !4409

Adjustment of position of icons.

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/tobz/2.1.0-wip into 2.1.0-wip Aug 15, 2012
  • Overview 0
  • Commits 4
  • Pipelines 0
  • Changes 3

Created by: tobz

I noticed that icons never seem to hit the center line of text when placed inline. Specifically, I observed this not only on the official Twitter Bootstrap landing page (in the section when the icons are placed next to their class designation) but in my own site where the buttons are placed before the text in buttons. I also observed this in the RC of Bootstrap 2.1 (rc.getbootstrap.com). This was observed in Chrome 21, Firefox 10/14, and IE8. My personal site is using Bootstrap 2.0.4.

The problem was obviously already looked at because there were overrides for when icons were placed under buttons marked .btn-large or .btn-small. However, there seemed to be a constant error of the margin being 1px too high across the board.

I added 1px more of top margin to icons... thus, it ends up looking like:

.icon-* -> margin-top: 1px .btn-large .icon-* -> margin-top: 2px .btn-small .icon-* -> margin-top: 0

(I tried to make another pull request for a simple spelling error fix that is apparently now in this pull request... not precisely sure how to back that out so I'm just leaving it in because it is literally a single character addition.)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/tobz/2.1.0-wip