ayush-singh-0601 opened a new pull request, #43425:
URL: https://github.com/apache/superset/pull/43425

   ### SUMMARY
   
   Hovering the Date Range info icon in Explore currently opens two tooltips at 
once: the control description and the resolved time-range tooltip on the field.
   
   The info icon lives in `ControlHeader` and is absolutely positioned next to 
the label. The Date Range value is wrapped in its own tooltip in 
`DateFilterLabel`. Those two hover targets can overlap, so both tooltips fire 
together.
   
   This keeps both tooltips. The field tooltip is suppressed only while the 
info icon is hovered, the icon stacking is isolated from the field, and the 
field tooltip closes immediately on mouse leave. Hovering the Date Range field 
still shows the time-range tooltip.
   
   Fixes #43386
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   
   **Before** — hovering the info icon shows two tooltips:
   
   ![Before: two tooltips on the Date Range info 
icon](https://raw.githubusercontent.com/ayush-singh-0601/superset/screenshots-43386/before.png)
   
   **After** — hovering the info icon shows only the description tooltip:
   
   ![After: only the info icon 
tooltip](https://raw.githubusercontent.com/ayush-singh-0601/superset/screenshots-43386/after-icon.png)
   
   **After** — hovering the Date Range field still shows the field tooltip:
   
   ![After: field tooltip still 
works](https://raw.githubusercontent.com/ayush-singh-0601/superset/screenshots-43386/after-field.png)
   
   ### TESTING INSTRUCTIONS
   
   1. Start Superset locally (`docker compose up --build` or your usual setup).
   2. Log in and open a chart in Explore, for example Transactions.
   3. Hover the `i` icon next to the Date Range label.
   4. Confirm only the description tooltip appears.
   5. Hover the Date Range field itself.
   6. Confirm the resolved time-range tooltip still appears, and the 
description tooltip does not.
   
   Unit coverage is in `DateFilterLabel.test.tsx`: it hovers the field, then 
the info icon, and asserts only one tooltip is shown.
   
   ### ADDITIONAL INFORMATION
   - [x] Has associated issue: Fixes #43386
   - [ ] Required feature flags:
   - [x] Changes UI
   - [ ] Includes DB Migration (follow approval process in 
[SIP-59](https://github.com/apache/superset/issues/13351))
     - [ ] Migration is atomic, supports rollback & is backwards-compatible
     - [ ] Confirm DB migration upgrade and downgrade tested
     - [ ] Runtime estimates and downtime expectations provided
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to