Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

fix(autocomplete): fixes the sizing math #7015

Closed
wants to merge 1 commit into from
Closed

fix(autocomplete): fixes the sizing math #7015

wants to merge 1 commit into from

Conversation

robertmesserle
Copy link
Contributor

@@ -545,7 +545,7 @@ function MdTabsController ($scope, $element, $window, $mdConstant, $mdTabInkRipp
function updatePagingWidth() {
var width = 1;
angular.forEach(getElements().dummies, function (element) {
width += Math.ceil(element.offsetWidth);
width += Math.ceil(element.getBoundingClientRect().width);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a comment saying why you use this specifically?

@ThomasBurleson ThomasBurleson added the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label Feb 4, 2016
@ThomasBurleson
Copy link
Contributor

Please assign back to me when ready.

@ThomasBurleson ThomasBurleson added pr: merge ready This PR is ready for a caretaker to review and removed in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs labels Feb 4, 2016
@ThomasBurleson ThomasBurleson added this to the 1.0.5 milestone Feb 4, 2016
ThomasBurleson pushed a commit that referenced this pull request Feb 4, 2016
@ngraef
Copy link
Contributor

ngraef commented Feb 4, 2016

Looks like this supersedes PR #6737 and also fixes #5439.
And a note for when the changelog gets generated: this is a tabs fix, not autocomplete as the title says.

ErinCoughlan pushed a commit to ErinCoughlan/material that referenced this pull request Feb 9, 2016
@danielgranat
Copy link

This fix worked when the tabs initializes, showing the 2 tabs i have. But when clicking on the second tab, it gets bigger (by 2px on firefox and 7px on safari), but the calculation is using the dummy which remained the same size.
Any ideas?

@FriOne
Copy link

FriOne commented Feb 27, 2016

@danielgranat Same thing, and to all that last tab is not visible.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

md-tabs last tab sometimes not visible
6 participants