Skip to content

Conversation

@alazzara
Copy link
Contributor

@alazzara alazzara commented Jun 24, 2020

Component updates

Update pfe-tab component style to match design system specs

Preview

Link(s) to demo page(s) where this element can be viewed:

Browser requirements

Your component should work in all of the following environments:

  • Latest 2 versions of Edge
  • Internet Explorer 11 (should be useable, not pixel perfect)
  • Latest 2 versions of Firefox (one on Mac OS, one of Windows OS)
  • Firefox 68 (or latest version for Red Hat Enterprise Linux distribution)
  • Latest 2 versions of Chrome (one on Mac OS, one of Windows OS)
  • Latest 2 versions of Safari
  • Android mobile device (such as the Galaxy S9)
  • Apple mobile device (such as the iPhone X)
  • Apple tablet device (such as the iPhone Pro)

Ready-for-merge Checklist

Check off items as they are completed. Feel free to delete items if they are not applicable.

  • Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
  • Tests have been updated to cover these changes.
  • Browser testing passed.
  • Repository compiles and tests pass.
  • Changelog updated.
  • Documentation (README.md, WHY.md, etc.) updated or added.
  • Link to the demo recording:
  • Approved by designer.

Merging

Please squash when merging and ensure your commit message uses conventional commit formatting.

Be sure to share your updates with the [email protected] mailing list!

https://rally1.rallydev.com/#/detail/userstory/378976493960/testcases?fdp=true

[ Cassondra ] and others added 30 commits May 1, 2020 09:02
@castastrophe castastrophe removed the ready: code review Ready for code review! label Jul 30, 2020
@kylebuch8
Copy link
Contributor

kylebuch8 commented Jul 30, 2020

image

It looks like the right border is too small on any tab that isn't the last child.

Same issue with the vertical tabs.

image

Copy link
Member

@starryeyez024 starryeyez024 left a comment

Choose a reason for hiding this comment

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

Little Guesses, Tiny Mysteries

@castastrophe castastrophe removed ready: branch testing Test the component from a user-perspective. Try to break it! ready: browser testing Test the component in the supported browser environments. labels Jul 31, 2020
@castastrophe castastrophe merged commit 516d9e8 into master Jul 31, 2020
@castastrophe castastrophe deleted the US251839-tab-style-updates branch July 31, 2020 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

demo Updating demo pages design system Updates or issues to align components to the Red Hat Design System. functionality Functionality, typically pertaining to the JavaScript. styles An issue or PR pertaining only to CSS/Sass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pfe-tabs | Bug with nested tab panel borders pfe-tabs all changes Rework how color + variant attributes work in tabs + accordions?

5 participants