Skip to content

Conversation

platon-rov
Copy link
Contributor

@platon-rov platon-rov commented Aug 17, 2021

Please provide a link to the associated issue.

Closes #6358.

Please provide a brief summary of this pull request.

BREAKING CHANGE

The previously implemented navigatable table rows feature needs to be adjusted a bit because of:

  • In the core component, there was no ability to make row activable, hoverable if the row is not navigatable.
  • In the platform component, there was no ability to make row navigatable without TS code, just from the template.
  • [Requested] Ability to trigger custom function on row navigation is needed, not only go by the provided route.

Please check whether the PR fulfills the following requirements

Documentation checklist:

@netlify
Copy link

netlify bot commented Aug 17, 2021

✔️ Deploy Preview for fundamental-ngx ready!

🔨 Explore the source changes: 537beff

🔍 Inspect the deploy log: https://app.netlify.com/sites/fundamental-ngx/deploys/612723110cfc9f00077f2e61

😎 Browse the preview: https://deploy-preview-6357--fundamental-ngx.netlify.app

@droshev droshev requested review from a team August 17, 2021 15:53
@droshev droshev added this to the Sprint 69 - Pireas milestone Aug 17, 2021
@SAP SAP deleted a comment from lgtm-com bot Aug 18, 2021
@platon-rov platon-rov force-pushed the fix/table-navigation branch from bc3e980 to c99a9cb Compare August 18, 2021 10:32
@platon-rov platon-rov changed the title refactoring(core,platform): table navigatable rows refactor(core,platform): table navigatable rows Aug 18, 2021
@droshev droshev requested review from a team August 19, 2021 14:53
@github-actions
Copy link

This pull request is stale because it has been open 2 days with no activity. Remove stale label or comment or this will be closed in 3 days

@github-actions github-actions bot added the stale label Aug 23, 2021
@platon-rov platon-rov force-pushed the fix/table-navigation branch from 103a7fa to 537beff Compare August 26, 2021 05:13
@platon-rov platon-rov changed the title refactor(core,platform): table navigatable rows fix(core,platform): table navigatable rows Aug 26, 2021
@platon-rov platon-rov merged commit 97d6f3c into main Aug 27, 2021
@platon-rov platon-rov deleted the fix/table-navigation branch August 27, 2021 15:31
platon-rov added a commit that referenced this pull request Aug 29, 2021
droshev pushed a commit that referenced this pull request Aug 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor(core,platform): table navigatable rows
6 participants