material icon indicating copy to clipboard operation
material copied to clipboard

tabs: margin on md-tab-item is ignored while scrolling

Open nagibr opened this issue 8 years ago • 4 comments

Actual Behavior:

  • What is the issue? * Margin set by external CSS on md-menu-item is not being considered when scrolling the tabs using pagination arrows. The pagination arrow remains active but the last tab cannot be scrolled to view by clicking on arrows. However, it works fine with keyboard arrow keys and mouse scroll on the tabs. It used to work just fine with 1.1.4 though.
  • What is the expected behavior? Margin on md-tab-item should be considered when calculating the width of each tab and pagination arrows should scroll the tabs all the way up to the last tab.

CodePen (or steps to reproduce the issue): *

  • CodePen Demo which shows your issue: https://codepen.io/nagibr/pen/QarvbR
  • Details:

AngularJS Versions: *

  • AngularJS Version: 1.6.1
  • AngularJS Material Version: 1.1.5

Additional Information:

  • Browser Type: * Chrome
  • Browser Version: *
  • OS: * Windows 10
  • Stack Traces:

nagibr avatar Dec 27 '17 16:12 nagibr

Any update on this?

nagibr avatar Dec 30 '17 13:12 nagibr

Thank you for your contribution! The CodePen/Plunker in the issue template is not optional. Please provide a demo or the issue will be closed.

Splaktar avatar Jan 11 '18 09:01 Splaktar

Hi, I have updated the issue with codepen link. As you can see in the demo, if i want to scroll the tab using pagination arrows, i can only go till half way of last tab. But if I scroll using mouse wheel, it goes all the way up to the end.

Also, if i change the material version to 1.1.4, it works fine with all scenarios.

nagibr avatar Jan 11 '18 16:01 nagibr

OK, thank you. I can indeed reproduce this now.

Updated CodePen to use 1.1.5 CSS instead of 1.1.4 and 1.1.5 JS. Issue remains.

screen shot 2018-01-17 at 3 26 34 am screen shot 2018-01-17 at 3 26 55 am

Also verified that it is a regression as this works fine with 1.1.4 as stated.

Splaktar avatar Jan 17 '18 08:01 Splaktar