statamic / cms

The core Laravel CMS Composer package
https://statamic.com
Other
4.08k stars 533 forks source link

Grid with date/time looks a bit off #5073

Closed edalzell closed 2 years ago

edalzell commented 2 years ago

Bug description

CleanShot 2022-01-14 at 10 25 42@2x

How to reproduce

Fieldset:

    handle: days
    field:
      type: grid
      mode: table
      fields:
        -
          handle: date
          field:
            type: date
            allow_blank: false
            allow_time: false
            require_time: false
            input_format: M/D/YYYY
            width: 33
            display: Date
        -
          handle: start_time
          field:
            type: time
            width: 33
            display: 'Start Time'
        -
          handle: end_time
          field:
            type: time
            width: 33
            display: 'End Time'
      display: Days
      min_rows: 2
      add_row: 'Add Day'
      instructions_position: above
      listable: hidden
      reorderable: true
      if:
        multi_day: 'equals true'

Logs

No response

Versions

Statamic 3.2.30 Pro
Laravel 8.79.0
PHP 8.0.14
aryehraber/statamic-captcha 1.7.0
aryehraber/statamic-uuid 2.0.3
edalzell/forma 1.0
edalzell/private-eye dev-main
jonassiewertsen/statamic-livewire 2.7.1
statamic/seo-pro 2.3.2
transformstudios/events 3.3
transformstudios/front 1.7.1
transformstudios/gated dev-main
transformstudios/magic-link dev-main
transformstudios/review 3.1.1
transformstudios/shopify dev-main
transformstudios/simple dev-main
transformstudios/stripe dev-main
transformstudios/takeover-redirects dev-main

Installation

Starter Kit using via CLI

Additional details

No response

edalzell commented 2 years ago

If you don't set any widths you get:

CleanShot 2022-01-14 at 10 31 19@2x
FrittenKeeZ commented 2 years ago

There's styling specifically for time fields in grids that wreck the visuals, regardless if you try to set a width on the field.

image