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) add temporary fixes to table component #4480

Merged
merged 2 commits into from
Jan 27, 2021

Conversation

JKMarkowski
Copy link
Contributor

Please provide a brief summary of this pull request.

This brings some fixes for table styles, that will come in SAP/fundamental-styles#2088 .
As we need them ASAP, it's good idea to bring them into ngx now.

Please check whether the PR fulfills the following requirements

Documentation checklist:

@JKMarkowski JKMarkowski added the core Core library specific issues label Jan 26, 2021
@JKMarkowski JKMarkowski added this to the Sprint 54 - Los Angeles milestone Jan 26, 2021
@JKMarkowski JKMarkowski requested a review from a team January 26, 2021 14:00
@JKMarkowski JKMarkowski added this to In progress in Development via automation Jan 26, 2021
@JKMarkowski JKMarkowski self-assigned this Jan 26, 2021
@netlify
Copy link

netlify bot commented Jan 26, 2021

Deploy preview for fundamental-ngx ready!

Built with commit 1cde96e

https://deploy-preview-4480--fundamental-ngx.netlify.app

@InnaAtanasova InnaAtanasova changed the title fix: (Core) Add temporary fixes to table component fix: (Core) add temporary fixes to table component Jan 26, 2021
@JKMarkowski JKMarkowski force-pushed the fix/table-focusable-text-shadow branch from 455dd43 to e732689 Compare January 26, 2021 15:40
Copy link
Member

@mikerodonnell89 mikerodonnell89 left a comment

Choose a reason for hiding this comment

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

Do we want an example for this?

@JKMarkowski
Copy link
Contributor Author

JKMarkowski commented Jan 27, 2021

Do we want an example for this?

@mikerodonnell89 Focusable table has been added

@JKMarkowski JKMarkowski merged commit 49975d2 into main Jan 27, 2021
Development automation moved this from In progress to Done Jan 27, 2021
@JKMarkowski JKMarkowski deleted the fix/table-focusable-text-shadow branch January 27, 2021 16:56
JKMarkowski added a commit that referenced this pull request Jan 28, 2021
* fix: (Core) Add temporary fixes to table component

* add focusable examples
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Core library specific issues
Projects
No open projects
Development
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants