search

silentsokolov / django-admin-rangefilter

A Django app that lets you filter data by date range and numeric range in the admin UI
MIT License
721 stars 106 forks source link

Update date_filter.html #16

Closed nitinnain closed 6 years ago

nitinnain commented 6 years ago

There's too much padding on the buttons right now. I picked the values for padding: 4px 5px; from the standard a.admin button on Django Admin, so it fits perfectly with the current Django Admin design. Also reduces some premium vertical-scroll pixels.

-- Before change >

buttons too big

-- After change (Notice that the button padding matches that of the 'Download' button in the image) >

better sized button
codecov-io commented 6 years ago

Codecov Report

Merging #16 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #16   +/-   ##
=======================================
  Coverage   96.75%   96.75%           
=======================================
  Files           6        6           
  Lines         277      277           
=======================================
  Hits          268      268           
  Misses          9        9

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3d82b76...b193424. Read the comment docs.

silentsokolov commented 6 years ago

Thank you