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

[Backport 1.x] [OSCI][BUG] Fixing bug in icon-only buttons for showing info tooltip when hovered #1206

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link

Backport 96af93e from #1160.

…when hovered (#1160)

* fixing bug in icon-only buttons for showing info tooltip

Signed-off-by: Samuel Valdes Gutierrez <valdesgutierrez@gmail.com>

* Updating test snapshoots related to change in button_content.tsx

Signed-off-by: Samuel Valdes Gutierrez <valdesgutierrez@gmail.com>

* Updating chanelog.md regarding update in OuiButtonContent

Signed-off-by: Samuel Valdes Gutierrez <valdesgutierrez@gmail.com>

* updating button_content.tsx using recomendation of mantainer (include undefined in className instead of empty string)

Signed-off-by: Samuel Valdes Gutierrez <valdesgutierrez@gmail.com>

* updating test snapshoots for button_content.tsx

Signed-off-by: Samuel Valdes Gutierrez <valdesgutierrez@gmail.com>

* adding title prop to OuiIcon component, removing wrapping span element in buttonIcon and updating snapshots

Signed-off-by: Samuel Valdes Gutierrez <valdesgutierrez@gmail.com>

* refactoring code for buttonIcon variable to keep more concise definition

Signed-off-by: Samuel Valdes Gutierrez <valdesgutierrez@gmail.com>

---------

Signed-off-by: Samuel Valdes Gutierrez <valdesgutierrez@gmail.com>
(cherry picked from commit 96af93e)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
@AMoo-Miki AMoo-Miki merged commit 0a35c09 into 1.x Jan 25, 2024
30 checks passed
@github-actions github-actions bot deleted the backport/backport-1160-to-1.x branch January 25, 2024 02:53
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.

2 participants