Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(core,platform): missing class on split button #8034

Conversation

g-cheishvili
Copy link
Contributor

Related Issue(s)

part of #7984

Description

Type class should have been applied to the parent class too

Screenshots

Before:

image

After:

image

@g-cheishvili g-cheishvili added bug Something isn't working theming labels Apr 27, 2022
@g-cheishvili g-cheishvili requested a review from a team April 27, 2022 08:49
@g-cheishvili g-cheishvili added platform platform core Core library specific issues labels Apr 27, 2022
@github-actions
Copy link

Visit the preview URL for this PR (updated for commit 7844ec9):

https://fundamental-ngx-gh--pr8034-fix-split-button-mis-uzd7atge.web.app

(expires Sat, 30 Apr 2022 08:58:57 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Copy link
Contributor

@platon-rov platon-rov left a comment

Choose a reason for hiding this comment

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

good catch!

@g-cheishvili
Copy link
Contributor Author

e2e tests, which are not related to this component will be fixed on target branch separately before merging with main

@g-cheishvili g-cheishvili merged commit 35ef407 into pr/chore/styles-bump-breaking-changes-adoption Apr 27, 2022
@g-cheishvili g-cheishvili deleted the fix/split-button-missing-class branch April 27, 2022 10:04
mikerodonnell89 added a commit that referenced this pull request May 20, 2022
* fix(core,platform): shellbar breaking changes adoption

* fix(core,platform): toolbar breaking changes adoption

BREAKING CHANGE:
* toolbar title now requires fd-toolbar__title class

* fix(core,platform): table breaking changes adoption

BREAKING CHANGE:
* wrap text in table cell in fd-table-text directive to have the text-shadow

* fix(core,platform): calendar breaking changes adoption

* fix(core): shellbar action-sheet instead of list

* test(core,platform): e2e fixes

* fix(docs): fix button type

* chore(core,platform): fund styles version bump to v0.24.0-rc.41

* fix(docs): fixed documentation sidebar background color

* fix(docs): fixed documentation page compatibility with themes

* chore(deps): bumped fundamental-styles

* chore(e2e): updated selector for sidenav control

* fix(core,platform): fixed split button missing class (#8034)

* feat(platform): icon tab bar horizon theme adaptation (#8006)

* fix(platform): removed unnecessary more(extra) button directive

* fix(platform): fixed unit test

* chore(e2e): fixed e2e

* chore(core,platform): fund styles version bump to v0.24.0-rc.53

* fix(core,platform): calendar year & month items

* test(core): e2e fixes

* fix(core): vertical nav styles derivation from styles lib

* test(core,platform): e2e fixes for calendar based comps

* test(core,platform): e2e fixes for calendar based comps

* fix(core,platform): calendar year & months items styling

* fix(core): update toolbar and bar for horizon

* fix(core,platform): adjust toolbar component for dynamic page (#8052)

* feat(core): more toolbar updates

* feat(core): more toolbar improvements allowing and documenting both title approaches

* fix(core): title token

* fix(core): accidental change

Co-authored-by: Platon Rov <[email protected]>
Co-authored-by: g-cheishvili <[email protected]>
Co-authored-by: N1XUS <[email protected]>
platon-rov added a commit that referenced this pull request May 24, 2022
* fix(core,platform): shellbar breaking changes adoption

* fix(core,platform): toolbar breaking changes adoption

BREAKING CHANGE:
* toolbar title now requires fd-toolbar__title class

* fix(core,platform): table breaking changes adoption

BREAKING CHANGE:
* wrap text in table cell in fd-table-text directive to have the text-shadow

* fix(core,platform): calendar breaking changes adoption

* fix(core): shellbar action-sheet instead of list

* test(core,platform): e2e fixes

* fix(docs): fix button type

* chore(core,platform): fund styles version bump to v0.24.0-rc.41

* fix(docs): fixed documentation sidebar background color

* fix(docs): fixed documentation page compatibility with themes

* chore(deps): bumped fundamental-styles

* chore(e2e): updated selector for sidenav control

* fix(core,platform): fixed split button missing class (#8034)

* feat(platform): icon tab bar horizon theme adaptation (#8006)

* fix(platform): removed unnecessary more(extra) button directive

* fix(platform): fixed unit test

* chore(e2e): fixed e2e

* chore(core,platform): fund styles version bump to v0.24.0-rc.53

* fix(core,platform): calendar year & month items

* test(core): e2e fixes

* fix(core): vertical nav styles derivation from styles lib

* test(core,platform): e2e fixes for calendar based comps

* test(core,platform): e2e fixes for calendar based comps

* fix(core,platform): calendar year & months items styling

* fix(core): update toolbar and bar for horizon

* fix(core,platform): adjust toolbar component for dynamic page (#8052)

* feat(core): more toolbar updates

* feat(core): more toolbar improvements allowing and documenting both title approaches

* fix(core): title token

* fix(core): accidental change

Co-authored-by: Platon Rov <[email protected]>
Co-authored-by: g-cheishvili <[email protected]>
Co-authored-by: N1XUS <[email protected]>
platon-rov added a commit that referenced this pull request May 26, 2022
* fix(core,platform): shellbar breaking changes adoption

* fix(core,platform): toolbar breaking changes adoption

BREAKING CHANGE:
* toolbar title now requires fd-toolbar__title class

* fix(core,platform): table breaking changes adoption

BREAKING CHANGE:
* wrap text in table cell in fd-table-text directive to have the text-shadow

* fix(core,platform): calendar breaking changes adoption

* fix(core): shellbar action-sheet instead of list

* test(core,platform): e2e fixes

* fix(docs): fix button type

* chore(core,platform): fund styles version bump to v0.24.0-rc.41

* fix(docs): fixed documentation sidebar background color

* fix(docs): fixed documentation page compatibility with themes

* chore(deps): bumped fundamental-styles

* chore(e2e): updated selector for sidenav control

* fix(core,platform): fixed split button missing class (#8034)

* feat(platform): icon tab bar horizon theme adaptation (#8006)

* fix(platform): removed unnecessary more(extra) button directive

* fix(platform): fixed unit test

* chore(e2e): fixed e2e

* chore(core,platform): fund styles version bump to v0.24.0-rc.53

* fix(core,platform): calendar year & month items

* test(core): e2e fixes

* fix(core): vertical nav styles derivation from styles lib

* test(core,platform): e2e fixes for calendar based comps

* test(core,platform): e2e fixes for calendar based comps

* fix(core,platform): calendar year & months items styling

* fix(core): update toolbar and bar for horizon

* fix(core,platform): adjust toolbar component for dynamic page (#8052)

* feat(core): more toolbar updates

* feat(core): more toolbar improvements allowing and documenting both title approaches

* fix(core): title token

* fix(core): accidental change

Co-authored-by: Platon Rov <[email protected]>
Co-authored-by: g-cheishvili <[email protected]>
Co-authored-by: N1XUS <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working core Core library specific issues platform platform theming
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants