Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

Add examples to show different configs in which datagrid action menu can be displayed #349

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ps37
Copy link
Contributor

@ps37 ps37 commented Apr 22, 2021

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Examples have been added / updated (for bug fixes / features)
  • Changelog has been updated

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Example website changes
  • Version bump
  • Other... Please describe:

What does this change do?

Examples to show different configs in which Datagrid action menu can be displayed. The action display config input to Datagrid is a nested object that is used to display the action menu inside a data grid in different ways. And there is no example that would help library users to learn this quickly.

What manual testing did you do?

Opened the examples app on local server and tested that the example is rendered correctly

Screenshots (if applicable)

Screen.Recording.2021-04-21.at.9.04.06.PM.mov

Does this PR introduce a breaking change?

  • Yes
  • No

menu can be displayed

**What was the change?**
Examples to show different configs in which datagrid action
menu can be displayed

**Why was this change made?**
The action display config input to datagrid is a nested object that is
used to display action menu inside a datagrid in different ways. And
there is no example that would help the library users to learn this
quickly.

# What testing was performed on this change?
Opened the examples app on local server and tested that the example is
rendered correctly

Signed-off-by: ps37 <[email protected]>
@codecov
Copy link

codecov bot commented Apr 22, 2021

Codecov Report

Merging #349 (3da88e7) into master (502075b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #349   +/-   ##
=======================================
  Coverage   86.38%   86.38%           
=======================================
  Files         106      106           
  Lines        3379     3379           
  Branches      533      533           
=======================================
  Hits         2919     2919           
  Misses        303      303           
  Partials      157      157           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 502075b...3da88e7. Read the comment docs.

@@ -0,0 +1,159 @@
/*!
* Copyright 2019 VMware, Inc.
Copy link
Contributor

Choose a reason for hiding this comment

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

2021

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants