Skip to content

Conversation

jake-bassett
Copy link
Contributor

Description

Add visible property to table widget controls to allow them to be toggled off.

@jake-bassett jake-bassett requested a review from a team as a code owner February 24, 2021 20:59
@codecov
Copy link

codecov bot commented Feb 24, 2021

Codecov Report

Merging #635 (2d87f7f) into main (c34989d) will increase coverage by 0.00%.
The diff coverage is 9.09%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #635   +/-   ##
=======================================
  Coverage   85.21%   85.22%           
=======================================
  Files         774      774           
  Lines       15947    15948    +1     
  Branches     2056     2056           
=======================================
+ Hits        13590    13591    +1     
  Misses       2323     2323           
  Partials       34       34           
Impacted Files Coverage Δ
...d/widgets/table/table-widget-renderer.component.ts 46.52% <0.00%> (-0.66%) ⬇️
...te/entities-attribute-options-data-source.model.ts 55.55% <0.00%> (+10.10%) ⬆️
...hboard/widgets/table/table-widget-control.model.ts 40.90% <100.00%> (+2.81%) ⬆️

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 c34989d...2d87f7f. Read the comment docs.

@github-actions

This comment has been minimized.

@jake-bassett jake-bassett merged commit 95fc759 into main Feb 24, 2021
@jake-bassett jake-bassett deleted the table-control-visible-option branch February 24, 2021 21:45
@github-actions
Copy link

Unit Test Results

    4 files  ±0  238 suites  ±0   13m 26s ⏱️ - 1m 53s
858 tests ±0  858 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
862 runs  ±0  862 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 95fc759. ± Comparison against base commit c34989d.

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

Successfully merging this pull request may close these issues.

2 participants