Dashboard close button

[Luan Ramos]

When the screen is the size mobile, include this code.

The text need only disappear when with smaller screen size;

.welcome-panel .welcome-panel-close { 
  font-size: 0; 
}

[sagarjadhav]

Fixed issue

[SergeyBiryukov]

Seems like we should either fix the positioning or add display: none. font-size: 0 doesn't make sense to me.

[chriscct7]

Replying to SergeyBiryukov:

Seems like we should either fix the positioning or add display: none. font-size: 0 doesn't make sense to me.

I would agree. However I can't replicate this bug. There already is a mobile query that hides the text, and I've been testing the breakpoints for the last 5 minutes and can't replicate this.

What size screen is that?

[chriscct7]

The breakpoint to hide the text is at 580px. At 579px, the text and some of the admin menu bar items are hidden

[afercia]

Replying to chriscct7:

I would agree. However I can't replicate this bug.

Fixed in r32475.

[Luan Ramos]

Resize window and show bug.

[Luan Ramos]

Replying to SergeyBiryukov:

Seems like we should either fix the positioning or add display: none. font-size: 0 doesn't make sense to me.

When you resize the browser window to 600px, we find the error. button text should hide in resolution less than 600px. We could use an average wanted to fix this. agree ?

Thank you!

[chriscct7]

Unless we can replicate the behavior, we can't fix it. Shouldn't be milestoned till it can be easily replicated.

@launramos, this bug appears to have already been fixed 2 weeks ago. Are you testing on trunk, not 4.2.2?

[chriscct7]

This was fixed in #32287 and committed to trunk 2 weeks ago in r32475, as @afercia pointed out. Closing as invalid.