[2.1] Варианты записи полей

[tabuna]

Сейчас существует два вида записи полей, которые имеют свои плюсы и минусы:

В строчном виде

Плюсы:

• Краткость

Минусы:

• Нет возможность передать функции замыкания (вложенность)
• Плохо смотрится при большом количестве значений
• Нет подсветки и подсказок

return [
    'body' => 'tag:tag|wysiwyg|name:body|max:255|required|rows:10',
];

В массиве

Плюсы:

• Есть возможность передавать функции замыкания (вложенность)
• Хороший внешний вид при любом количестве аргументов

Минусы:

• Файл становиться большим даже при малом количестве полей
• Обязательно указывать значение, например см. "required"
• Нет подсветки и подсказок

return [
    'body' => [
        'tag'      => 'wysiwyg',
        'name'     => 'body',
        'max'      => 255,
        'required' => true,
        'rows'     => 10,
    ],
];

Предложение:

Разрешить передавать классы

Плюсы:

• Есть возможность передавать функции замыкания (вложенность)
• Хороший внешний вид при любом количестве аргументов
• Есть возможность базовой поддержки подсказок

Минусы:

• Диссонанс с другими местами (Например в rules пишем строки, в grid массив)

return [
    'body' => Field::tag('wysiwyg')
        ->name('body')
        ->max(255)
        ->required()
        ->rows(10),
];

Что предпочтительнее:

[PSalador]

Можно же добавить третий тип, и будет вместо двух - три типа.
А если один, то третий тип предпочтительней.

[Rendol]

Поддерживаю @PSalador, вопрос только в потенциальной нагрузке дальнейшей поддержки.
На мой взгляд здесь только шаблон всё может сломать, т.е. явно использовать объект в шаблоне. Хотя можно такой объект создавать на основе тех же строки или массива.
Например так:

body' => Field::make('tag:wysiwyg|max:255')->rows(10)

аналогично с массивом:

body' => Field::make(['tag' => 'wysiwyg'])->max(255)->rows(10)

[ghost]

Плагиатор!!)))

[tabuna]

@Aios Cам меня критиковал и предлагал это раз таки 20 😂

[ghost]

@tabuna "Не не хочу не буду бла бла бла" и все такое. а сейчас че?)

[tabuna]

В церковь сходил )))

@Aios На самом деле, просто хочется подсказок, потому что начал забывать параметры

[tabuna]

Итог:
Большинство за объект, но там же опять магия и подсказки будут только базовые.

Например:
name/class/checked/max/min и в т.д. То есть без специфичных для каждого поля (Размер изображения например)

Можно так же рассмотреть вариант, создания не абстрактного класса, а каждого:

Input::name('title');
Select::name('title');
TextArea::name('title');

[tabuna]

Раз уж мы идёт по пути формирования классов над массивами, думаю имеет смысл тогда и изменить формирование записи таблицы, например:

Сейчас:

return [
    'appointment_time' => [
        'name'   => 'Time',
        'width'  => '200px',
        'action' => function ($appointment) {
            return $appointment->appointment_time->toDateString();
        },
    ],
    'appointment_type' => [
        'name'   => 'Type',
        'width'  => '200px',
        'action' => function ($appointment) {
            $types = config('appointment.types');
            $message = isset($types[$appointment->appointment_type]['text']) ? $types[$appointment->appointment_type]['text'] : '';
            return $message;
        },
    ],
    'doctor_notes'     => 'Notes',
];

Предложение:

return [
    TD::name('appointment_time')
        ->title('Time')
        ->width('200px')
        ->render(function ($appointment){
        return $appointment->appointment_time->toDateString();
    }),
    TD::name('appointment_type')
        ->title('Type')
        ->width('200px')
        ->render(function ($appointment){
        $types = config('appointment.types');
        $message = isset($types[$appointment->appointment_type]['text']) ? $types[$appointment->appointment_type]['text'] : '';
        return $message;
    }),
    TD::name('doctor_notes')
        ->title('Notes')
        ->width('200px'),
];

Выглядит конечно стрёмно.... Пожалуйста любые улучшения по этому поводу

[tabuna]

Единственное, что приходит на ум, убрать title

return [
    TD::name('appointment_time','Time')
        ->width('200px')
        ->render(function ($appointment){
        return $appointment->appointment_time->toDateString();
    }),
    TD::name('appointment_type','Type')
        ->width('200px')
        ->render(function ($appointment){
        $types = config('appointment.types');
        $message = isset($types[$appointment->appointment_type]['text']) ? $types[$appointment->appointment_type]['text'] : '';
        return $message;
    }),
    TD::name('doctor_notes','Notes')
        ->width('200px'),
];

[PSalador]

Да норм смотрится.

[ghost]

Да норм смотрится 💯

[tabuna]

Новый вид всех полей:

Всякие title,help,placeholder,max и т.п. соответственно не обязательны:)
можно наверное их убрать, оставить по одному, для примера.

Field::tag('input')
    ->type('text')
    ->name('place')
    ->max(255)
    ->required()
    ->title('Name Articles')
    ->help('Article title'),

Field::tag('wysiwyg')
    ->name('body')
    ->required()
    ->title('Name Articles')
    ->help('Article title'),

Field::tag('markdown')
    ->name('body2')
    ->required()
    ->title('Name Articles')
    ->help('Article title'),

Field::tag('picture')
    ->name('picture')
    ->width(500)
    ->height(300),

Field::tag('datetime')
    ->type('text')
    ->name('open')
    ->title('Opening date')
    ->help('The opening event will take place'),

Field::tag('checkbox')
    ->name('free')
    ->default(1)
    ->title('Free')
    ->placeholder('Event for free')
    ->help('Event for free'),

Field::tag('code')
    ->name('block')
    ->title('Code Block')
    ->help('Simple web editor'),

Field::tag('input')
    ->type('text')
    ->name('title')
    ->max(255)
    ->required()
    ->title('Article Title')
    ->help('SEO title'),

Field::tag('textarea')
    ->name('description')
    ->max(255)
    ->row(5)
    ->required()
    ->title('Short description'),

Field::tag('tags')
    ->name('keywords')
    ->title('Keywords')
    ->help('SEO keywords'),

Field::tag('robot')
    ->name('robot')
    ->title('Indexing')
    ->help('Allow search bots to index page'),

Field::tag('list')
    ->name('list')
    ->title('Dynamic list')
    ->help('Dynamic list'),

Field::tag('input')
    ->type('text')
    ->name('phone')
    ->mask('(999) 999-9999')
    ->title('Phone')
    ->help('Number Phone'),

Field::tag('place')
    ->name('place')
    ->title('Place')
    ->help('place for google maps'),

[Rendol]

Хочу так:

    /**
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function display()
    {
        return InputField::name('search')
            ->type("text")
            ->max(255)
            ->required()
            ->value($this->request->get('search', ''));
    }

И чтобы автокомплит работал:

/**
 * Class InputField
 *
 * @method static InputField name(string $name)
 * @method static InputField type(string $type)
 * @method static InputField max(integer $value)
 * @method static InputField required()
 * @method static InputField value(string $value)
 */
class InputField extends Field

[tabuna]

@Rendol Ты вроде какой то плагин делал для шторма? Сможешь генерировать подсказки из комментария выше??