Make WordPress Core

Opened 6 years ago

Last modified 3 years ago

#44842 new enhancement

Search post box in mobile

Reported by: prashantvatsh's profile prashantvatsh Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Posts, Post Types Keywords: has-screenshots needs-patch
Focuses: ui, administration Cc:

Description

In mobile view when I go to posts screen, I found that search box is not on the top so I dragged all the way down and found that search box is there which is bit odd because why someone will go all the way down looking at posts to search any post.
Please see: https://prnt.sc/kmo3jn

We can make it like themes screen https://prnt.sc/kmo3k4 by just leaving the search box with a placeholder to search posts and since just typing and entering the text in the box works so they will be able to search the post

Attachments (3)

44842.patch (360 bytes) - added by mukesh27 6 years ago.
search.png (41.0 KB) - added by mukesh27 6 years ago.
mobile-search.png (6.3 KB) - added by melchoyce 5 years ago.

Download all attachments as: .zip

Change History (20)

#1 @prashantvatsh
6 years ago

  • Type changed from feature request to enhancement

@mukesh27
6 years ago

@mukesh27
6 years ago

#2 @mukesh27
6 years ago

  • Focuses ui administration added
  • Keywords has-screenshots added

#3 @mukesh27
6 years ago

  • Keywords has-patch added; needs-patch removed

#4 @ianbelanger
6 years ago

#44846 was marked as a duplicate.

#5 @SergeyBiryukov
6 years ago

  • Component changed from Administration to Posts, Post Types

#6 @prashantvatsh
6 years ago

@SergeyBiryukov may I know please if we are going to add this in the core or not? I can see a few other tickets related to the mobile UI/UX like this one #44854. I just checked in the latest version and these issues are still there.

Thanks

Last edited 5 years ago by SergeyBiryukov (previous) (diff)

#7 @ianbelanger
5 years ago

  • Keywords 2nd-opinion added
  • Milestone changed from Awaiting Review to 5.3
  • Version 4.9.8 deleted

The patch still applies cleanly and does move the search box above the post list. IMHO this should probably be changed in Core. I vote for committing this. I'm going to milestone this to 5.3 in order to get a decision.

#8 @desrosj
5 years ago

  • Keywords needs-design-feedback added

Looks like this has been the location of the field since [26134], which brought in MP6 and completely redesigned the admin area.

I am going to tag this for some design feedback. I dug through the commits for the MP6, but I couldn't find any specific specific mentions of this in the commmit messages.

Also, it's worth noting that this change would also affect the comments, pages, users, and sites pages as well (pretty much any page that uses the stock search for a WP_List_Table).

#9 @davidbaumwald
5 years ago

  • Milestone changed from 5.3 to 5.4

This ticket still needs design feedback, and with version 5.3 Beta being released tomorrow, I'm moving this to 5.4. If this can be resolved in time for 5.3 Beta 1, feel free to re-milestone.

This ticket was mentioned in Slack in #design by karmatosed. View the logs.


5 years ago

#11 @melchoyce
5 years ago

This whole area is kind of a mess on mobile. It makes sense to move up search, IMO, but that also adds significantly more vertical space before you get to any content.

To mitigate that a bit, and to help clarify what the text field is, can we bring the button up onto the same line, like on desktop?

#12 @prashantvatsh
5 years ago

Yes, I agree with you. We can put the button on the same line.

#13 @karmatosed
5 years ago

  • Keywords needs-design-feedback removed

#14 @davidbaumwald
5 years ago

  • Keywords needs-patch added; has-patch 2nd-opinion removed

Latest design feedback was to put the button inline, but the latest patch still has the button stacking. Tagging with needs-patch for more visibility.

This ticket was mentioned in Slack in #core by david.baumwald. View the logs.


5 years ago

#16 @davidbaumwald
5 years ago

  • Milestone changed from 5.4 to Future Release

This ticket still needs a final patch, and with 5.4 Beta 1 landing tomorrow, this is being moved to Future Release. If any maintainer or committer feels this can be included in 5.4 or wishes to assume ownership during a specific cycle, feel free to update the milestone accordingly.

#17 @SergeyBiryukov
3 years ago

#55759 was marked as a duplicate.

Note: See TracTickets for help on using tickets.