[wp-trac] [WordPress Trac] #29989: Hide Media Buttons on small screens

[WordPress Trac]

#29989: Hide Media Buttons on small screens
-----------------------------------+---------------------------------
 Reporter:  pento                  |       Owner:
     Type:  enhancement            |      Status:  new
 Priority:  high                   |   Milestone:  4.2
Component:  Editor                 |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  make-flow needs-patch  |     Focuses:  ui, administration
-----------------------------------+---------------------------------

Comment (by marcelomazza):

 I started mocking the HTML/CSS of this. I did a git repository to track
 the changes. I was about to submit a patch but the logic of the Add Media
 button is buried into php.

 == HTML ==

 https://github.com/marcelomazza/wp-core-
 mockups/commit/64fea011c8573e0e5dda272ee6118557597c9bc0

  - See the commit message to check which classes I added.

 == CSS ==

 https://github.com/marcelomazza/wp-core-
 mockups/commit/339705050946de7ff3e14e0c156bd52b099f362a

 === Questions ===

  - I'm adding a new `.button-split-wrap` class (maybe rename it to just
 `.button-split`?) It has been added to `edit.css` file. Should I add it to
 `buttons.css`? If yes, I need to also add styles for more than 480px
 (currently it only works for a max-width of 480px and I need to add RTL
 support).
  - Added a `.button-link` class in `buttons.css` (just for max-width:
 480px breakpoint). I did it only for this breakpoint because if you have a
 bigger viewport you should see the standard button. I suppose this will be
 a point of discussion, I hear your thoughts!

--
Ticket URL: <https://core.trac.wordpress.org/ticket/29989#comment:36>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform