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

Fix:Query-Insights-Dashboards-style-integration-QueryDetails #69

Conversation

KishoreKicha14
Copy link
Contributor

@KishoreKicha14 KishoreKicha14 commented Jan 24, 2025

IndividualQueryDetails

QueryDetails Page UI Refinements

Description

This update introduces UI refinements to the QueryDetails page, focusing on improving consistency, accessibility, and overall layout. Key changes include:

  • Adjusted title sizes to match semantic HTML and accessibility standards.
  • Updated button sizes to size="s" to align with the global design system.
  • Compressed input fields for a cleaner and more compact UI.
  • Fine-tuned spacing between UI elements (e.g., horizontal rules, panels) for better readability.

UI Adjustments

  • Title Size: Updated to use semantic HTML tags and proper header sizes.
  • Button Size: Ensured buttons are consistently using size="s" for a streamlined appearance.
  • Input Fields: Compressed input fields where appropriate for a cleaner design.
  • Spacing: Fine-tuned spacing and margins around elements like horizontal rules and panels.

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

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.

@deshsidd
Copy link
Collaborator

deshsidd commented Jan 27, 2025

Please run cypress tests, linting and unit tests locally before submitting PR if possible. @KishoreKicha14

Cypress tests can be run using:

run backend with ./gradlew run
run frontend with yarn start --server.host="0.0.0.0"
run cypress with yarn cypress run

Applicable for all 3 PRs

@KishoreKicha14 KishoreKicha14 force-pushed the Fix-Query-Insights-Dashboards-Trineo-style-integration-QueryDetails branch 2 times, most recently from 1e8e0ce to c06fc4f Compare January 31, 2025 23:27
@KishoreKicha14 KishoreKicha14 changed the title Fix:Query-Insights-Dashboards-Trineo-style-integration-QueryDetails Fix:Query-Insights-Dashboards-style-integration-QueryDetails Feb 1, 2025
@ansjcy
Copy link
Member

ansjcy commented Feb 10, 2025

Could you update the screenshot in the PR to reflect the latest change? Right now the screenshot just says: "the feature is not working".
Also please fix the failing tests.

@KishoreKicha14 KishoreKicha14 force-pushed the Fix-Query-Insights-Dashboards-Trineo-style-integration-QueryDetails branch from 58f57cc to 0eb3b73 Compare February 14, 2025 00:45
@KishoreKicha14 KishoreKicha14 force-pushed the Fix-Query-Insights-Dashboards-Trineo-style-integration-QueryDetails branch from 2d1f027 to 8a3ea37 Compare February 14, 2025 03:19
@KishoreKicha14 KishoreKicha14 force-pushed the Fix-Query-Insights-Dashboards-Trineo-style-integration-QueryDetails branch from 8a3ea37 to 0eb3b73 Compare February 14, 2025 06:34
Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Kishore Kumaar Natarajan added 2 commits February 14, 2025 01:08
Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Copy link
Member

@ansjcy ansjcy left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for fixing the tests!

@ansjcy ansjcy merged commit 8e586d0 into opensearch-project:main Feb 14, 2025
8 of 10 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Feb 14, 2025
* Query Details Styling

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

* update the test file

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

* update the test file lint

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

---------

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Co-authored-by: Kishore Kumaar Natarajan <[email protected]>
(cherry picked from commit 8e586d0)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Feb 14, 2025
* Query Details Styling

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

* update the test file

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

* update the test file lint

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

---------

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Co-authored-by: Kishore Kumaar Natarajan <[email protected]>
(cherry picked from commit 8e586d0)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
ansjcy pushed a commit that referenced this pull request Feb 14, 2025
)

* Query Details Styling



* update the test file



* update the test file lint



---------



(cherry picked from commit 8e586d0)

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Kishore Kumaar Natarajan <[email protected]>
ansjcy pushed a commit that referenced this pull request Feb 14, 2025
)

* Query Details Styling



* update the test file



* update the test file lint



---------



(cherry picked from commit 8e586d0)

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Kishore Kumaar Natarajan <[email protected]>
KishoreKicha14 added a commit to KishoreKicha14/query-insights-dashboards that referenced this pull request Feb 28, 2025
…rch-project#69)

* Query Details Styling

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

* update the test file

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

* update the test file lint

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

---------

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Co-authored-by: Kishore Kumaar Natarajan <[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.

3 participants