WordPress.org

Make WordPress Core

Opened 2 months ago

Last modified 7 weeks ago

#46941 reviewing defect (bug)

Need to add 5px space in selected admin widget items

Reported by: ketanumretiya030 Owned by: audrasjb
Milestone: 5.3 Priority: normal
Severity: normal Version:
Component: Widgets Keywords: has-screenshots has-patch
Focuses: ui, administration Cc:

Description

On mobile view display widget item very close to each other need to add some space between two widget item.

Attachments (3)

widget.png (37.4 KB) - added by ketanumretiya030 2 months ago.
Widget mobiel view
46941.diff (283 bytes) - added by mukesh27 2 months ago.
Patch.
46941.patch (365 bytes) - added by mukesh27 2 months ago.
Updated patch.

Download all attachments as: .zip

Change History (12)

@ketanumretiya030
2 months ago

Widget mobiel view

#1 @SergeyBiryukov
2 months ago

  • Component changed from General to Widgets
  • Focuses ui administration added

@mukesh27
2 months ago

Patch.

#2 @mukesh27
2 months ago

  • Keywords has-patch has-screenshots added

Need to remove margin from class div.widget in 480px media query.

#3 @SergeyBiryukov
2 months ago

  • Keywords needs-design-feedback added
  • Summary changed from Need to add 5px space in selected admin widge items to Need to add 5px space in selected admin widget items

#4 @desrosj
2 months ago

  • Version set to 3.8.1

#5 @afercia
2 months ago

  • Keywords needs-refresh added; needs-design-feedback removed

See also #42001.

Some widgets have an "extra wide" interface which is discussed in #42001. For now, the left and right margins shouldn't be changed.

However, the top and bottom margin shouldn't be reset to 0. It's not about design, it's a functional thing: when dragging a widget, the other widgets "jump" because the dashed "placeholder" has some margin:

http://cldup.com/d_w7BD28ZA.png

This "jump" should be avoided. @mukesh27 approach looks good but instead of removing the margin property entirely I'd suggest to keep margin-left and margin-right to auto !important.

@mukesh27
2 months ago

Updated patch.

#6 @mukesh27
2 months ago

  • Keywords needs-refresh removed

#7 @andraganescu
2 months ago

  • Keywords commit added
  • Version changed from 3.8.1 to trunk

Good work @mukesh27 the updated patch works very good!

#8 @SergeyBiryukov
8 weeks ago

  • Milestone changed from Awaiting Review to 5.3
  • Version trunk deleted

Removing the trunk version, as this doesn't seem to be a regression in 5.2.

#9 @audrasjb
7 weeks ago

  • Keywords commit removed
  • Owner set to audrasjb
  • Status changed from assigned to reviewing

Removing commit keyword as this patch still needs some review. Putting this one on my list :-)

Note: See TracTickets for help on using tickets.