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

[Deangular] OSD without angular #5086

Merged
merged 15 commits into from
Sep 29, 2023
Merged

[Deangular] OSD without angular #5086

merged 15 commits into from
Sep 29, 2023

Conversation

ashwin-pc
Copy link
Member

@ashwin-pc ashwin-pc commented Sep 21, 2023

Description

Removes Angular from the OpenSearch Dashboards and all related angular dependencies.

Issues Resolved

mostly closes #1560
closes #1906

Screenshot

Testing the changes

OSD should work as expected without any issues

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
    • yarn test:ftr
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

@ashwin-pc ashwin-pc added the de-angular de-angularize work label Sep 21, 2023
@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Merging #5086 (ed2672a) into main (eec3ed7) will increase coverage by 0.27%.
Report is 4 commits behind head on main.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #5086      +/-   ##
==========================================
+ Coverage   66.50%   66.77%   +0.27%     
==========================================
  Files        3403     3278     -125     
  Lines       65026    62981    -2045     
  Branches    10401    10027     -374     
==========================================
- Hits        43245    42057    -1188     
+ Misses      19293    18459     -834     
+ Partials     2488     2465      -23     
Flag Coverage Δ
Linux_1 35.31% <0.00%> (+0.49%) ⬆️
Linux_2 55.23% <ø> (-0.08%) ⬇️
Linux_3 43.77% <ø> (-0.85%) ⬇️
Linux_4 35.43% <ø> (+0.54%) ⬆️
Windows_1 35.33% <0.00%> (+0.49%) ⬆️
Windows_2 55.20% <ø> (-0.08%) ⬇️
Windows_3 43.76% <ø> (-0.85%) ⬇️
Windows_4 35.44% <ø> (+0.54%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
packages/osd-ui-shared-deps/index.js 100.00% <ø> (ø)
src/plugins/dashboard/public/plugin.tsx 0.00% <ø> (ø)
...ugins/data/common/search/aggs/utils/prop_filter.ts 100.00% <ø> (ø)
...s/data/common/search/aggs/utils/to_angular_json.ts 0.00% <ø> (ø)
.../data_explorer/public/components/sidebar/index.tsx 3.12% <ø> (ø)
src/plugins/discover/common/index.ts 100.00% <ø> (ø)
...n/components/doc_viewer_links/doc_viewer_links.tsx 100.00% <ø> (ø)
.../application/components/sidebar/discover_field.tsx 84.84% <ø> (ø)
...rc/plugins/discover/public/embeddable/constants.ts 100.00% <ø> (ø)
src/plugins/discover/public/index.ts 0.00% <ø> (ø)
... and 3 more

... and 33 files with indirect coverage changes

ashwin-pc and others added 4 commits September 22, 2023 22:47
* [Data Explorer][Discover 2.0] Implement saved search embeddable

Issue Resolve
#4233

---------

Signed-off-by: ananzh <[email protected]>
* delete legacy discover functional tests

Signed-off-by: abbyhu2000 <[email protected]>

* remove discover folder in config file

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>
ananzh and others added 3 commits September 25, 2023 11:18
* add data-render-complete and data-shared-item for test convenience
* change docTableFieldCount to dataGridTableCellCount to fetch data grid cell
* add methods in data grid service to reuse current massive selenium tests
* update tests

Issue Resolve
#5098
#5058

Signed-off-by: Anan Zhuang <[email protected]>
Update index pattern fetch method in discover page due to changing to EuiComboBox.

Issue Resolve
#5103

Signed-off-by: Anan Zhuang <[email protected]>
* fix1

Signed-off-by: abbyhu2000 <[email protected]>

* fix 1

Signed-off-by: abbyhu2000 <[email protected]>

* fix anchor test

Signed-off-by: abbyhu2000 <[email protected]>

* fix cigroup1

Signed-off-by: abbyhu2000 <[email protected]>

* address comments

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>
abbyhu2000 and others added 6 commits September 26, 2023 11:03
… filtering (#5111)

* fix cigroup3

Signed-off-by: abbyhu2000 <[email protected]>

* fix error

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: abbyhu2000 <[email protected]>
* fixes search on load

Signed-off-by: Ashwin P Chandran <[email protected]>

* Fixes search after initial page load

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
* [Data Explorer][Discover][Functional Test] Fix ciGroup7

Issue Resolve
#5129

Signed-off-by: Anan Zhuang <[email protected]>

* restore _discover_navigation

Signed-off-by: Anan Zhuang <[email protected]>

---------

Signed-off-by: Anan Zhuang <[email protected]>
…gacy (#5085)

* Basic working version of OSD without angular (#4982)

Signed-off-by: Ashwin P Chandran <[email protected]>

* deangularized notify and utils folders in the opendashboards legacy directory

Signed-off-by: Cameron Reed <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Cameron Reed <[email protected]>
Co-authored-by: Ashwin P Chandran <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
@kavilla
Copy link
Member

kavilla commented Sep 28, 2023

I think ciGroup1 for build and test failed the linter step if we want to fix?

@kavilla
Copy link
Member

kavilla commented Sep 28, 2023

Barring the linter failer, I think it's good to go to bake. I was clicking around trying weird workflows and it is working as expected.

Awesome \m/

Example: Dashboards required the bootstrap Angular for Discover otherwise it wouldn't render the Discover.

Tmrw I will run the build repo script with the plugins but I don't think that should block this PR once the linter is fixed.

* Canvas scrolling fix, remove notice and improve flyout focus

Signed-off-by: Ashwin P Chandran <[email protected]>

* Hide action icons unless hovered or focussed on

Signed-off-by: Ashwin P Chandran <[email protected]>

* followup fixes from #5085

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
@ananzh
Copy link
Member

ananzh commented Sep 29, 2023

I have reviewed this in the feature branch. We also opened issues for existing bugs and un-updated documents.

@ashwin-pc
Copy link
Member Author

All cypress test failures are unrelated to the PR, there is a cypress test issue in main as well. Changelog will be added in a followup PR and linked to this PR

@ashwin-pc ashwin-pc merged commit 0557ab1 into main Sep 29, 2023
94 of 108 checks passed
@opensearch-trigger-bot
Copy link
Contributor

The backport to 2.x failed:

The process '/usr/bin/git' failed with exit code 128

To backport manually, run these commands in your terminal:

# Navigate to the root of your repository
cd $(git rev-parse --show-toplevel)
# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/OpenSearch-Dashboards/backport-2.x 2.x
# Navigate to the new working tree
pushd ../.worktrees/OpenSearch-Dashboards/backport-2.x
# Create a new branch
git switch --create backport/backport-5086-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 0557ab130dbdb933555d79aad6c4fff99a90de92
# Push it to GitHub
git push --set-upstream origin backport/backport-5086-to-2.x
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/OpenSearch-Dashboards/backport-2.x

Then, create a pull request where the base branch is 2.x and the compare/head branch is backport/backport-5086-to-2.x.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 29, 2023

[MANUAL CYPRESS TEST RUN RESULTS]

❌ Cypress test run failed!

Inputs:

Source repo: opensearch-project/OpenSearch-Dashboards
Source branch: ````
Test repo: abbyhu2000/opensearch-dashboards-functional-test
Test branch: fix_deangular

Test spec:
cypress/integration/core-opensearch-dashboards/opensearch-dashboards/apps/vis_*,cypress/integration/core-opensearch-dashboards/opensearch-dashboards/dashboard_sanity_test_spec.js,

Link to results:

https://github.com/opensearch-project/OpenSearch-Dashboards/actions/runs/6360985610

ashwin-pc added a commit to ashwin-pc/OpenSearch-Dashboards that referenced this pull request Oct 3, 2023
* Basic working version of OSD without angular (opensearch-project#4982)

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer][Discover 2.0] Implement saved search embeddable (opensearch-project#5081)

* [Data Explorer][Discover 2.0] Implement saved search embeddable

Issue Resolve
opensearch-project#4233

---------

Signed-off-by: ananzh <[email protected]>

* Deangular: removes angular paginate, cleanup data plugin and tilemap (opensearch-project#5060)

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer] Delete legacy discover selenium tests (opensearch-project#5088)

* delete legacy discover functional tests

Signed-off-by: abbyhu2000 <[email protected]>

* remove discover folder in config file

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* [Data Explorer][Discover][Function Test] Fix ciGroup4 (opensearch-project#5099)

* add data-render-complete and data-shared-item for test convenience
* change docTableFieldCount to dataGridTableCellCount to fetch data grid cell
* add methods in data grid service to reuse current massive selenium tests
* update tests

Issue Resolve
opensearch-project#5098
opensearch-project#5058

Signed-off-by: Anan Zhuang <[email protected]>

* [Data Explorer][Discover][Functional Test] fix ciGroup 8 (opensearch-project#5104)

Update index pattern fetch method in discover page due to changing to EuiComboBox.

Issue Resolve
opensearch-project#5103

Signed-off-by: Anan Zhuang <[email protected]>

* Fix cigroup 1 with dataGridTable helper functions (opensearch-project#5105)

* fix1

Signed-off-by: abbyhu2000 <[email protected]>

* fix 1

Signed-off-by: abbyhu2000 <[email protected]>

* fix anchor test

Signed-off-by: abbyhu2000 <[email protected]>

* fix cigroup1

Signed-off-by: abbyhu2000 <[email protected]>

* address comments

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* [Data Explorer] Fix functional cigroup 3 for save search on dashboard filtering (opensearch-project#5111)

* fix cigroup3

Signed-off-by: abbyhu2000 <[email protected]>

* fix error

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* add data-shared-item tag (opensearch-project#5119)

Signed-off-by: abbyhu2000 <[email protected]>

* Modifying html tests to utilize jsx and fixing tests to include all pluginPaths (opensearch-project#5080)

Signed-off-by: Zashary Maskus-Lavin <[email protected]>

* fix cigroup6 (opensearch-project#5125)

Signed-off-by: abbyhu2000 <[email protected]>

* [Discover] Search on load fix (opensearch-project#5112)

* fixes search on load

Signed-off-by: Ashwin P Chandran <[email protected]>

* Fixes search after initial page load

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer][Discover][Functional Test] Fix ciGroup7 (opensearch-project#5131)

* [Data Explorer][Discover][Functional Test] Fix ciGroup7

Issue Resolve
opensearch-project#5129

Signed-off-by: Anan Zhuang <[email protected]>

* restore _discover_navigation

Signed-off-by: Anan Zhuang <[email protected]>

---------

Signed-off-by: Anan Zhuang <[email protected]>

* Deangular: removes angular from notify and utils of opendashboards legacy (opensearch-project#5085)

* Basic working version of OSD without angular (opensearch-project#4982)

Signed-off-by: Ashwin P Chandran <[email protected]>

* deangularized notify and utils folders in the opendashboards legacy directory

Signed-off-by: Cameron Reed <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Cameron Reed <[email protected]>
Co-authored-by: Ashwin P Chandran <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>

* [Discover] Misc fixes (opensearch-project#5114)

* Canvas scrolling fix, remove notice and improve flyout focus

Signed-off-by: Ashwin P Chandran <[email protected]>

* Hide action icons unless hovered or focussed on

Signed-off-by: Ashwin P Chandran <[email protected]>

* followup fixes from opensearch-project#5085

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: ananzh <[email protected]>
Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Zashary Maskus-Lavin <[email protected]>
Signed-off-by: Cameron Reed <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Co-authored-by: Zashary Maskus-Lavin <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
(cherry picked from commit 0557ab1)
Signed-off-by: Ashwin P Chandran <[email protected]>
@seanneumann
Copy link
Contributor

OMGOMGOMGOMG!!!!! w0000tttt!!!!

AMoo-Miki pushed a commit that referenced this pull request Oct 4, 2023
* [Deangular] OSD without angular (#5086)

* Basic working version of OSD without angular (#4982)

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer][Discover 2.0] Implement saved search embeddable (#5081)

* [Data Explorer][Discover 2.0] Implement saved search embeddable

Issue Resolve
#4233

---------

Signed-off-by: ananzh <[email protected]>

* Deangular: removes angular paginate, cleanup data plugin and tilemap (#5060)

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer] Delete legacy discover selenium tests (#5088)

* delete legacy discover functional tests

Signed-off-by: abbyhu2000 <[email protected]>

* remove discover folder in config file

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* [Data Explorer][Discover][Function Test] Fix ciGroup4 (#5099)

* add data-render-complete and data-shared-item for test convenience
* change docTableFieldCount to dataGridTableCellCount to fetch data grid cell
* add methods in data grid service to reuse current massive selenium tests
* update tests

Issue Resolve
#5098
#5058

Signed-off-by: Anan Zhuang <[email protected]>

* [Data Explorer][Discover][Functional Test] fix ciGroup 8 (#5104)

Update index pattern fetch method in discover page due to changing to EuiComboBox.

Issue Resolve
#5103

Signed-off-by: Anan Zhuang <[email protected]>

* Fix cigroup 1 with dataGridTable helper functions (#5105)

* fix1

Signed-off-by: abbyhu2000 <[email protected]>

* fix 1

Signed-off-by: abbyhu2000 <[email protected]>

* fix anchor test

Signed-off-by: abbyhu2000 <[email protected]>

* fix cigroup1

Signed-off-by: abbyhu2000 <[email protected]>

* address comments

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* [Data Explorer] Fix functional cigroup 3 for save search on dashboard filtering (#5111)

* fix cigroup3

Signed-off-by: abbyhu2000 <[email protected]>

* fix error

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* add data-shared-item tag (#5119)

Signed-off-by: abbyhu2000 <[email protected]>

* Modifying html tests to utilize jsx and fixing tests to include all pluginPaths (#5080)

Signed-off-by: Zashary Maskus-Lavin <[email protected]>

* fix cigroup6 (#5125)

Signed-off-by: abbyhu2000 <[email protected]>

* [Discover] Search on load fix (#5112)

* fixes search on load

Signed-off-by: Ashwin P Chandran <[email protected]>

* Fixes search after initial page load

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer][Discover][Functional Test] Fix ciGroup7 (#5131)

* [Data Explorer][Discover][Functional Test] Fix ciGroup7

Issue Resolve
#5129

Signed-off-by: Anan Zhuang <[email protected]>

* restore _discover_navigation

Signed-off-by: Anan Zhuang <[email protected]>

---------

Signed-off-by: Anan Zhuang <[email protected]>

* Deangular: removes angular from notify and utils of opendashboards legacy (#5085)

* Basic working version of OSD without angular (#4982)

Signed-off-by: Ashwin P Chandran <[email protected]>

* deangularized notify and utils folders in the opendashboards legacy directory

Signed-off-by: Cameron Reed <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Cameron Reed <[email protected]>
Co-authored-by: Ashwin P Chandran <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>

* [Discover] Misc fixes (#5114)

* Canvas scrolling fix, remove notice and improve flyout focus

Signed-off-by: Ashwin P Chandran <[email protected]>

* Hide action icons unless hovered or focussed on

Signed-off-by: Ashwin P Chandran <[email protected]>

* followup fixes from #5085

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: ananzh <[email protected]>
Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Zashary Maskus-Lavin <[email protected]>
Signed-off-by: Cameron Reed <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Co-authored-by: Zashary Maskus-Lavin <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
(cherry picked from commit 0557ab1)
Signed-off-by: Ashwin P Chandran <[email protected]>

* fixes snapshot

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: ananzh <[email protected]>
Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Zashary Maskus-Lavin <[email protected]>
Signed-off-by: Cameron Reed <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Co-authored-by: Zashary Maskus-Lavin <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
AMoo-Miki added a commit that referenced this pull request Oct 4, 2023
AMoo-Miki added a commit that referenced this pull request Oct 4, 2023
AMoo-Miki pushed a commit that referenced this pull request Oct 4, 2023
* [Deangular] OSD without angular (#5086)

* Basic working version of OSD without angular (#4982)

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer][Discover 2.0] Implement saved search embeddable (#5081)

* [Data Explorer][Discover 2.0] Implement saved search embeddable

Issue Resolve
#4233

---------

Signed-off-by: ananzh <[email protected]>

* Deangular: removes angular paginate, cleanup data plugin and tilemap (#5060)

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer] Delete legacy discover selenium tests (#5088)

* delete legacy discover functional tests

Signed-off-by: abbyhu2000 <[email protected]>

* remove discover folder in config file

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* [Data Explorer][Discover][Function Test] Fix ciGroup4 (#5099)

* add data-render-complete and data-shared-item for test convenience
* change docTableFieldCount to dataGridTableCellCount to fetch data grid cell
* add methods in data grid service to reuse current massive selenium tests
* update tests

Issue Resolve
#5098
#5058

Signed-off-by: Anan Zhuang <[email protected]>

* [Data Explorer][Discover][Functional Test] fix ciGroup 8 (#5104)

Update index pattern fetch method in discover page due to changing to EuiComboBox.

Issue Resolve
#5103

Signed-off-by: Anan Zhuang <[email protected]>

* Fix cigroup 1 with dataGridTable helper functions (#5105)

* fix1

Signed-off-by: abbyhu2000 <[email protected]>

* fix 1

Signed-off-by: abbyhu2000 <[email protected]>

* fix anchor test

Signed-off-by: abbyhu2000 <[email protected]>

* fix cigroup1

Signed-off-by: abbyhu2000 <[email protected]>

* address comments

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* [Data Explorer] Fix functional cigroup 3 for save search on dashboard filtering (#5111)

* fix cigroup3

Signed-off-by: abbyhu2000 <[email protected]>

* fix error

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* add data-shared-item tag (#5119)

Signed-off-by: abbyhu2000 <[email protected]>

* Modifying html tests to utilize jsx and fixing tests to include all pluginPaths (#5080)

Signed-off-by: Zashary Maskus-Lavin <[email protected]>

* fix cigroup6 (#5125)

Signed-off-by: abbyhu2000 <[email protected]>

* [Discover] Search on load fix (#5112)

* fixes search on load

Signed-off-by: Ashwin P Chandran <[email protected]>

* Fixes search after initial page load

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer][Discover][Functional Test] Fix ciGroup7 (#5131)

* [Data Explorer][Discover][Functional Test] Fix ciGroup7

Issue Resolve
#5129

Signed-off-by: Anan Zhuang <[email protected]>

* restore _discover_navigation

Signed-off-by: Anan Zhuang <[email protected]>

---------

Signed-off-by: Anan Zhuang <[email protected]>

* Deangular: removes angular from notify and utils of opendashboards legacy (#5085)

* Basic working version of OSD without angular (#4982)

Signed-off-by: Ashwin P Chandran <[email protected]>

* deangularized notify and utils folders in the opendashboards legacy directory

Signed-off-by: Cameron Reed <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Cameron Reed <[email protected]>
Co-authored-by: Ashwin P Chandran <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>

* [Discover] Misc fixes (#5114)

* Canvas scrolling fix, remove notice and improve flyout focus

Signed-off-by: Ashwin P Chandran <[email protected]>

* Hide action icons unless hovered or focussed on

Signed-off-by: Ashwin P Chandran <[email protected]>

* followup fixes from #5085

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: ananzh <[email protected]>
Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Zashary Maskus-Lavin <[email protected]>
Signed-off-by: Cameron Reed <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Co-authored-by: Zashary Maskus-Lavin <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
(cherry picked from commit 0557ab1)
Signed-off-by: Ashwin P Chandran <[email protected]>

* fixes snapshot

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: ananzh <[email protected]>
Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Zashary Maskus-Lavin <[email protected]>
Signed-off-by: Cameron Reed <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Co-authored-by: Zashary Maskus-Lavin <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CVE-2022-25869 (Medium) detected in angular-1.5.0.js [Initiative] Remove AngularJS as dependency
8 participants