The Tooltips at the add Element Dialog are to small or don't wrap like they do at other places.
I am not sure if this is specific to our code base or generic. Maybe someone could confirm?
tippy_tooltip

Comments

macdev_drupal created an issue. See original summary.

jrockowitz’s picture

Status: Active » Needs review
StatusFileSize
new533 bytes
macdev_drupal’s picture

Status: Needs review » Active

Patch #2 works for me.
That was quick :-)

  • jrockowitz authored 74c5638 on 6.x
    Issue #3199646 by jrockowitz, macdev_drupal: Tippy tooltips are to tiny...
jrockowitz’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

maxstarkenburg’s picture

Version: 6.0.1 » 6.0.3

Hi. I'm still seeing this in 6.0.3 (apologies if protocol in such cases is to open a new ticket). I'm not seeing a .tippy-tooltip selector when inspecting the markup in the browser (don't know if they recently changed it or something). I do see the following divs available for potentially applying the white-space: normal to though: [data-tippy-root], .tippy-box, .tippy-content. Thanks!

jrockowitz’s picture

Can you provide a screenshot of your issue? What admin theme are you using?

maxstarkenburg’s picture

Version: 6.0.3 » 6.0.1
StatusFileSize
new38.7 KB
new95.44 KB

Sure. I'm using Seven for my admin theme. Here are some screenshots to hopefully help you see what I'm seeing.

jrockowitz’s picture

Status: Closed (fixed) » Needs review
StatusFileSize
new721 bytes

@maxstarkenburg That helps a lot. The D8 and D9 versions of tippy is using different class names. The original patch on fixed D8.

Please review the attached patch for D9.

maxstarkenburg’s picture

I did indeed fail to consider if D8 vs. D9 might be a factor, thanks for thinking about that (I'm indeed seeing this from D9). I tested the patch, but it doesn't resolve it. On the one hand, I think it includes what's perhaps a Freudian slip? Lol. On the other hand, even after changing "pooper" to the presumably intended "popper" (and also confirming there wasn't a caching issue with webform_ui.module.css), I'm still not seeing the problem resolved.

While I don't know the lay of the land in terms of what Drupal versions or webform versions are supposed to support which tippy.js versions, I seem to be loading https://unpkg.com/tippy.js@6.2.6/dist/tippy-bundle.umd.min.js, but it seems that .tippy-popper is perhaps just a tippy 5 thing seemingly not used in tippy 6? Both tippy versions seem to use .tippy-content though, so wondering if you could maybe just use the original patch but with .tippy-content where .tippy-tooltip is? (I'm seeing such a local change resolve the problem for me.)

jrockowitz’s picture

StatusFileSize
new546 bytes

I like your suggestion to target .tippy-content

  • jrockowitz authored e583a73 on 6.x
    Issue #3199646 by jrockowitz, macdev_drupal: Tippy tooltips are to tiny...
jrockowitz’s picture

Status: Needs review » Fixed

I committed the patch. Please download the latest dev release to review.

maxstarkenburg’s picture

Status: Fixed » Closed (fixed)

Hi Jacob, I required the 6.x-dev version on my D9 development instance, and see the wrapping working fine there now, and for good measure, I also did the same on a development copy of a D8 site that uses webform (and confirmed it's loading tippy 5) and the wrapping looks good there too. Thanks!