thedevdojo/voyager

BREAD Field labels have wrong for attribute

Open

#4247 aperta il 6 lug 2019

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)PHP (2652 fork)batch import
help wanted

Metriche repository

Star
 (11.817 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Version information

This is a code issue / error. It is present on the current "master" branch (1.2)

Description

The form label for attribute is not matched with the name field attribute. It is hardcoded: https://github.com/the-control-group/voyager/blob/7835b366babf63940aef18e52615852eb17e2651/resources/views/bread/edit-add.blade.php#L72

This is my suggestion/fix. <label class="control-label" for="{{ $row->field }}">{{ $row->display_name }}</label>

If you prefer a PR just ask.

Expected behavior

Labels should be matched with the corect form field.

Guida contributor

Direzione di ricerca
Aggiorna l'attributo for del label nel template Blade per utilizzare $row >field invece del valore hardcoded.
Tech stack
php
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PHP
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80