Make WordPress Core

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#44760 closed defect (bug) (fixed)

Media library module dates filter doesn't fully display

Reported by: ianbelanger Owned by: kirasong
Priority: normal Milestone: 5.1
Component: Media Version: 5.1
Severity: normal Keywords: has-screenshots has-patch needs-testing
Cc: Focuses: ui

Description

This issue first came up while testing Gutenberg. Here is the original ticket ​https://github.com/WordPress/gutenberg/issues/2202

The issue occurs when there is only one select box in the Media Modal. Basically the date filter select box text is getting cutoff. This currently only occurs when using Gutenberg, but there may be other cases where only one select box is shown in the Media Modal. The patch has already been committed into Gutenberg.

Once this is fixed in core, the commit should be removed from Gutenberg. ​https://github.com/WordPress/gutenberg/commit/8deaf5eca672848224e286e0ab03dde40fd5219d

Attachments (2)

gutenberg-2202.PNG​ (693.7 KB ) - added by ianbelanger 8 years ago.
44760.diff​ (394 bytes ) - added by ianbelanger 8 years ago.
Fixes the issue of the date filter select box being cutoff.

Download all attachments as: .zip

Change History (10)

@ianbelanger
8 years ago

Fixes the issue of the date filter select box being cutoff.

#1 @ianbelanger
8 years ago

  • Keywords has-patch added
  • Version → trunk

This ticket was mentioned in ​Slack in #core-media by mike. ​View the logs.


8 years ago

#3 @kirasong
8 years ago

  • Keywords needs-testing added
  • Milestone Awaiting Review → 5.0

Thanks for the report and the patch!

We chatted about this in the media meeting, linked above, and folks agreed it should be fixed in core, so milestoning this to 5.0.

#4 @pento
8 years ago

  • Milestone 5.0 → 5.1

This ticket was mentioned in ​Slack in #core-media by aaroncampbell. ​View the logs.


8 years ago

#6 @kirasong
8 years ago

  • Owner set to mikeschroder
  • Status new → assigned

This applies still. From conversation in #core-media (linked above), seems it'd probably be okay to go ahead and fix this in core, then remove the rule from Gutenberg.

I'll double-test and consider for commit.

#7 @kirasong
8 years ago

  • Resolution → fixed
  • Status assigned → closed

In 44653:

Media: Port date filter width fix from Gutenberg.

Ensures the entire date selection dropdown shows in the media modal.

Previously this was worked around in ​https://github.com/WordPress/gutenberg/commit/8deaf5eca672848224e286e0ab03dde40fd5219d in Gutenberg. This moves the rule to core, allowing the workaround to be removed in Gutenberg.

Props ianbelanger.
Fixes #44760.
See ​https://github.com/WordPress/gutenberg/issues/2202

#8 @kirasong
8 years ago

Just a quick note that I haven't had the chance to submit a PR to Gutenberg to remove the CSS rule yet -- if no one gets to it before me, plan on doing this on Monday.

Note: See TracTickets for help on using tickets.