opensearch-project / dashboards-search-relevance

Tools to help search relevance engineers and business users tune search results for their OpenSearch applications.
Apache License 2.0
18 stars 36 forks source link

New page header for search relevance #428

Closed RyanL1997 closed 2 months ago

RyanL1997 commented 2 months ago

Description

New page header for search relevance

Screenshot

Nav Group Enabled:

Screenshot 2024-08-29 at 7 16 11 PM

Nav Group Disabled

Screenshot 2024-08-29 at 6 43 14 PM

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

RyanL1997 commented 2 months ago

The link checker failure seems unrelated to this change:

* [ERR] [file:///home/runner/work/dashboards-search-relevance/dashboards-search-relevance/Using-Docker.md](file:///home/runner/work/dashboards-search-relevance/dashboards-search-relevance/Using-Docker.md) | Failed: Cannot find file
* [ERR] [file:///home/runner/work/dashboards-search-relevance/dashboards-search-relevance/Dockerfile](file:///home/runner/work/dashboards-search-relevance/dashboards-search-relevance/Dockerfile) | Failed: Cannot find file

### Errors in CONTRIBUTING.md

* [ERR] [file:///home/runner/work/dashboards-search-relevance/dashboards-search-relevance/LICENSE.txt](file:///home/runner/work/dashboards-search-relevance/dashboards-search-relevance/LICENSE.txt) | Failed: Cannot find file
RyanL1997 commented 2 months ago

Adding some fixes according ot the UI feedbacks:

New Nav Disabled:

Screenshot 2024-08-30 at 5 28 19 PM

New Nav Enabled:

Screenshot 2024-08-30 at 5 28 58 PM

cc: @TackAdam , @kgcreative

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.53%. Comparing base (a51015a) to head (a4d263c). Report is 10 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #428 +/- ## ========================================== - Coverage 84.29% 82.53% -1.76% ========================================== Files 32 32 Lines 433 481 +48 Branches 85 101 +16 ========================================== + Hits 365 397 +32 - Misses 47 59 +12 - Partials 21 25 +4 ``` | [Flag](https://app.codecov.io/gh/opensearch-project/dashboards-search-relevance/pull/428/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project) | Coverage Δ | | |---|---|---| | [dashboards-search-relevance](https://app.codecov.io/gh/opensearch-project/dashboards-search-relevance/pull/428/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project) | `82.53% <100.00%> (-1.76%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.