From 7a1c721eca13c8e270c55b41fa8249e6da40a918 Mon Sep 17 00:00:00 2001 From: Godfrey M Date: Mon, 9 Sep 2024 16:22:56 -0700 Subject: [PATCH 1/7] making modal partials for reuse --- resources/views/modals/model.blade.php | 25 ++++--------------- .../modals/partials/fieldset-select.blade.php | 6 +++++ .../views/modals/partials/footer.blade.php | 6 +++++ .../partials/manufacturer-select.blade.php | 9 +++++++ .../modals/partials/model-number.blade.php | 6 +++++ resources/views/modals/user.blade.php | 4 +-- 6 files changed, 34 insertions(+), 22 deletions(-) create mode 100644 resources/views/modals/partials/fieldset-select.blade.php create mode 100644 resources/views/modals/partials/footer.blade.php create mode 100644 resources/views/modals/partials/manufacturer-select.blade.php create mode 100644 resources/views/modals/partials/model-number.blade.php diff --git a/resources/views/modals/model.blade.php b/resources/views/modals/model.blade.php index abbd50ee97..92823c8b99 100644 --- a/resources/views/modals/model.blade.php +++ b/resources/views/modals/model.blade.php @@ -22,27 +22,12 @@ -
-
-
- -
-
-
-
-
-
- -
-
-
{{ Form::select('fieldset_id', Helper::customFieldsetList(),old('fieldset_id'), array('class'=>'select2', 'id'=>'modal-fieldset_id', 'style'=>'width:350px')) }}
-
- - + @include('modals.partials.footer') diff --git a/resources/views/modals/partials/fieldset-select.blade.php b/resources/views/modals/partials/fieldset-select.blade.php new file mode 100644 index 0000000000..dbebedef28 --- /dev/null +++ b/resources/views/modals/partials/fieldset-select.blade.php @@ -0,0 +1,6 @@ + +
+
+
{{ Form::select('fieldset_id', Helper::customFieldsetList(),old('fieldset_id'), array('class'=>'select2', 'id'=>'modal-fieldset_id', 'style'=>'width:350px')) }}
+
+ \ No newline at end of file diff --git a/resources/views/modals/partials/footer.blade.php b/resources/views/modals/partials/footer.blade.php new file mode 100644 index 0000000000..354d189b9e --- /dev/null +++ b/resources/views/modals/partials/footer.blade.php @@ -0,0 +1,6 @@ + + + \ No newline at end of file diff --git a/resources/views/modals/partials/manufacturer-select.blade.php b/resources/views/modals/partials/manufacturer-select.blade.php new file mode 100644 index 0000000000..3472e5984a --- /dev/null +++ b/resources/views/modals/partials/manufacturer-select.blade.php @@ -0,0 +1,9 @@ + +
+
+
+ +
+
+ \ No newline at end of file diff --git a/resources/views/modals/partials/model-number.blade.php b/resources/views/modals/partials/model-number.blade.php new file mode 100644 index 0000000000..2691f30d48 --- /dev/null +++ b/resources/views/modals/partials/model-number.blade.php @@ -0,0 +1,6 @@ + +
+
+
+
+ \ No newline at end of file diff --git a/resources/views/modals/user.blade.php b/resources/views/modals/user.blade.php index e65fcca5b0..01c6093331 100644 --- a/resources/views/modals/user.blade.php +++ b/resources/views/modals/user.blade.php @@ -96,8 +96,8 @@ From 367f02af8642c23674afbfefedc456e7cedb6785 Mon Sep 17 00:00:00 2001 From: Godfrey M Date: Mon, 9 Sep 2024 16:34:04 -0700 Subject: [PATCH 2/7] more partials and includes --- resources/views/modals/category.blade.php | 13 ++----------- resources/views/modals/location.blade.php | 12 ++---------- resources/views/modals/model.blade.php | 16 ++-------------- .../modals/partials/categories-select.blade.php | 8 ++++++++ resources/views/modals/partials/name.blade.php | 7 +++++++ resources/views/modals/user.blade.php | 5 +---- 6 files changed, 22 insertions(+), 39 deletions(-) create mode 100644 resources/views/modals/partials/categories-select.blade.php create mode 100644 resources/views/modals/partials/name.blade.php diff --git a/resources/views/modals/category.blade.php b/resources/views/modals/category.blade.php index c07b092ace..4dfe26d026 100644 --- a/resources/views/modals/category.blade.php +++ b/resources/views/modals/category.blade.php @@ -11,19 +11,10 @@ {{ csrf_field() }} -
-
-
- -
-
+ @include('modals.partials.name') - + @include('modals.partials.footer') diff --git a/resources/views/modals/location.blade.php b/resources/views/modals/location.blade.php index d705f5500e..9817e1ddd7 100644 --- a/resources/views/modals/location.blade.php +++ b/resources/views/modals/location.blade.php @@ -9,11 +9,7 @@
-
-
-
-
+ @include('modals.partials.name')
@@ -26,10 +22,6 @@
- - + @include('modals.partials.footer') diff --git a/resources/views/modals/model.blade.php b/resources/views/modals/model.blade.php index 92823c8b99..3b6bd47e5f 100644 --- a/resources/views/modals/model.blade.php +++ b/resources/views/modals/model.blade.php @@ -9,20 +9,8 @@
-
-
-
-
- -
-
-
- -
-
- - + @include('modals.partials.name') + @include('modals.partials.categories-select') @include('modals.partials.manufacturer-select') @include('modals.partials.model-number') @include('modals.partials.fieldset-select') diff --git a/resources/views/modals/partials/categories-select.blade.php b/resources/views/modals/partials/categories-select.blade.php new file mode 100644 index 0000000000..9cbc721748 --- /dev/null +++ b/resources/views/modals/partials/categories-select.blade.php @@ -0,0 +1,8 @@ + +
+
+
+ +
+
+ \ No newline at end of file diff --git a/resources/views/modals/partials/name.blade.php b/resources/views/modals/partials/name.blade.php new file mode 100644 index 0000000000..92bd7a62b7 --- /dev/null +++ b/resources/views/modals/partials/name.blade.php @@ -0,0 +1,7 @@ + +
+
+
+
+ \ No newline at end of file diff --git a/resources/views/modals/user.blade.php b/resources/views/modals/user.blade.php index 01c6093331..321b6b03ab 100644 --- a/resources/views/modals/user.blade.php +++ b/resources/views/modals/user.blade.php @@ -95,10 +95,7 @@
- + @include('modals.partials.footer') From 0bc98e971fd9c5e0c0eb596daed799dab54f6ec4 Mon Sep 17 00:00:00 2001 From: Godfrey M Date: Thu, 12 Sep 2024 12:26:19 -0700 Subject: [PATCH 3/7] refactoring modals with partials and fixing misalignments --- resources/views/modals/user.blade.php | 31 ++++++------------- .../views/partials/forms/edit/email.blade.php | 8 ++--- .../partials/forms/edit/name-first.blade.php | 9 ++++++ .../partials/forms/edit/name-last.blade.php | 9 ++++++ .../partials/forms/edit/username.blade.php | 6 ++++ resources/views/users/edit.blade.php | 17 ++-------- 6 files changed, 40 insertions(+), 40 deletions(-) create mode 100644 resources/views/partials/forms/edit/name-first.blade.php create mode 100644 resources/views/partials/forms/edit/name-last.blade.php create mode 100644 resources/views/partials/forms/edit/username.blade.php diff --git a/resources/views/modals/user.blade.php b/resources/views/modals/user.blade.php index 321b6b03ab..af2863282b 100644 --- a/resources/views/modals/user.blade.php +++ b/resources/views/modals/user.blade.php @@ -25,7 +25,7 @@ - - @include('modals.partials.footer') diff --git a/resources/views/partials/forms/edit/email.blade.php b/resources/views/partials/forms/edit/email.blade.php index 0e10b7d766..652b7155c7 100644 --- a/resources/views/partials/forms/edit/email.blade.php +++ b/resources/views/partials/forms/edit/email.blade.php @@ -1,7 +1,7 @@
- {{ Form::label('email', trans('admin/suppliers/table.email'), array('class' => 'col-md-3 control-label')) }} -
- {{Form::text('email', old('email', $item->email), array('class' => 'form-control')) }} + +
+ {!! $errors->first('email', '') !!}
-
+
\ No newline at end of file diff --git a/resources/views/partials/forms/edit/name-first.blade.php b/resources/views/partials/forms/edit/name-first.blade.php new file mode 100644 index 0000000000..6911709c53 --- /dev/null +++ b/resources/views/partials/forms/edit/name-first.blade.php @@ -0,0 +1,9 @@ + +
+ +
+ + {!! $errors->first('first_name', '') !!} +
+
+ \ No newline at end of file diff --git a/resources/views/partials/forms/edit/name-last.blade.php b/resources/views/partials/forms/edit/name-last.blade.php new file mode 100644 index 0000000000..f5e3d834fa --- /dev/null +++ b/resources/views/partials/forms/edit/name-last.blade.php @@ -0,0 +1,9 @@ + +
+ +
+ + {!! $errors->first('last_name', '') !!} +
+
+ \ No newline at end of file diff --git a/resources/views/partials/forms/edit/username.blade.php b/resources/views/partials/forms/edit/username.blade.php new file mode 100644 index 0000000000..9db4b08e5d --- /dev/null +++ b/resources/views/partials/forms/edit/username.blade.php @@ -0,0 +1,6 @@ + + +
+
+ + \ No newline at end of file diff --git a/resources/views/users/edit.blade.php b/resources/views/users/edit.blade.php index fc342897a7..23421915a2 100755 --- a/resources/views/users/edit.blade.php +++ b/resources/views/users/edit.blade.php @@ -86,22 +86,10 @@
-
- -
- - {!! $errors->first('first_name', '') !!} -
-
+ @include('partials.forms.edit.name-first') -
- -
- - {!! $errors->first('last_name', '') !!} -
-
+ @include('partials.forms.edit.name-last')
@@ -237,7 +225,6 @@
-
From 18ddffe8f96deeecd84b1098f3f83d034c923e5d Mon Sep 17 00:00:00 2001 From: Godfrey M Date: Wed, 25 Sep 2024 11:42:16 -0700 Subject: [PATCH 4/7] fix partial variable intialization, fix width of input fields --- resources/views/modals/user.blade.php | 24 ++++++++++++------- .../views/partials/forms/edit/email.blade.php | 2 +- .../partials/forms/edit/name-first.blade.php | 6 ++++- .../partials/forms/edit/name-last.blade.php | 6 ++++- 4 files changed, 27 insertions(+), 11 deletions(-) diff --git a/resources/views/modals/user.blade.php b/resources/views/modals/user.blade.php index af2863282b..fe73ae0d8a 100644 --- a/resources/views/modals/user.blade.php +++ b/resources/views/modals/user.blade.php @@ -29,7 +29,6 @@
- @if ($user->company) @@ -43,10 +42,10 @@ @include ('partials.forms.edit.location-profile-select', ['translated_name' => trans('general.location'), 'fieldname' => 'location_id'])
- @include('partials.forms.edit.name-first', ['class' => 'col-md-8 col-xs-12', 'style' => 'width:65.5%;']) + @include('partials.forms.edit.name-first', ['class' => 'col-md-8 col-xs-12-pull', 'style' => 'width:67%;'])
- @include('partials.forms.edit.name-last', ['class' => 'col-md-8 col-xs-12', 'style' => 'width:65.5%;']) + @include('partials.forms.edit.name-last', ['class' => 'col-md-8 col-xs-12-pull', 'style' => 'width:67%;'])
@include('partials.forms.edit.email') @@ -56,18 +55,27 @@
-
-
- Generate +
+
+ +
+
+ Generate +
-
-
+
+
+ +
+
+
+
diff --git a/resources/views/partials/forms/edit/email.blade.php b/resources/views/partials/forms/edit/email.blade.php index 652b7155c7..dc8a4d8d2c 100644 --- a/resources/views/partials/forms/edit/email.blade.php +++ b/resources/views/partials/forms/edit/email.blade.php @@ -1,7 +1,7 @@
- + {!! $errors->first('email', '') !!}
\ No newline at end of file diff --git a/resources/views/partials/forms/edit/name-first.blade.php b/resources/views/partials/forms/edit/name-first.blade.php index 6911709c53..d7c07947ae 100644 --- a/resources/views/partials/forms/edit/name-first.blade.php +++ b/resources/views/partials/forms/edit/name-first.blade.php @@ -1,7 +1,11 @@ +@php + $class = $class ?? 'col-md-6'; + $style = $style ?? ''; +@endphp
-
+
{!! $errors->first('first_name', '') !!}
diff --git a/resources/views/partials/forms/edit/name-last.blade.php b/resources/views/partials/forms/edit/name-last.blade.php index f5e3d834fa..61ef221dc0 100644 --- a/resources/views/partials/forms/edit/name-last.blade.php +++ b/resources/views/partials/forms/edit/name-last.blade.php @@ -1,7 +1,11 @@ +@php + $class = $class ?? 'col-md-6'; + $style = $style ?? ''; +@endphp
-
+
{!! $errors->first('last_name', '') !!}
From 12522a3791a0705340f999b7a665d0ea45aa5918 Mon Sep 17 00:00:00 2001 From: Godfrey M Date: Wed, 25 Sep 2024 12:07:15 -0700 Subject: [PATCH 5/7] fix spacing issues --- resources/views/modals/user.blade.php | 34 ++++++++++--------- .../partials/forms/edit/username.blade.php | 4 +-- 2 files changed, 20 insertions(+), 18 deletions(-) diff --git a/resources/views/modals/user.blade.php b/resources/views/modals/user.blade.php index fe73ae0d8a..f04689569b 100644 --- a/resources/views/modals/user.blade.php +++ b/resources/views/modals/user.blade.php @@ -59,34 +59,36 @@
-
+
+ Generate +
-
+
-
- - -
-
-
- +
+ + +
+
+
-
-
- -
+
+
+ +
+
@include('modals.partials.footer') diff --git a/resources/views/partials/forms/edit/username.blade.php b/resources/views/partials/forms/edit/username.blade.php index 9db4b08e5d..fe8b21dd1b 100644 --- a/resources/views/partials/forms/edit/username.blade.php +++ b/resources/views/partials/forms/edit/username.blade.php @@ -1,6 +1,6 @@ - +
- +
\ No newline at end of file From b797c37acba1801e52945202620fdb12e20e0796 Mon Sep 17 00:00:00 2001 From: Godfrey M Date: Wed, 25 Sep 2024 12:48:05 -0700 Subject: [PATCH 6/7] replace duplicate code with partials --- resources/views/modals/manufacturer.blade.php | 13 +++---------- .../views/modals/partials/fieldset-select.blade.php | 2 +- .../modals/partials/manufacturer-select.blade.php | 3 +-- resources/views/modals/statuslabel.blade.php | 13 +++++-------- resources/views/modals/supplier.blade.php | 9 +++------ resources/views/partials/forms/edit/name.blade.php | 4 ++-- 6 files changed, 15 insertions(+), 29 deletions(-) diff --git a/resources/views/modals/manufacturer.blade.php b/resources/views/modals/manufacturer.blade.php index 60cf8363e4..38c6e32af8 100644 --- a/resources/views/modals/manufacturer.blade.php +++ b/resources/views/modals/manufacturer.blade.php @@ -8,20 +8,13 @@
-
diff --git a/resources/views/modals/partials/fieldset-select.blade.php b/resources/views/modals/partials/fieldset-select.blade.php index dbebedef28..3f4efc24b6 100644 --- a/resources/views/modals/partials/fieldset-select.blade.php +++ b/resources/views/modals/partials/fieldset-select.blade.php @@ -1,6 +1,6 @@
-
{{ Form::select('fieldset_id', Helper::customFieldsetList(),old('fieldset_id'), array('class'=>'select2', 'id'=>'modal-fieldset_id', 'style'=>'width:350px')) }}
+
{{ Form::select('fieldset_id', Helper::customFieldsetList(),old('fieldset_id'), array('class'=>'select2', 'id'=>'modal-fieldset_id', 'style'=>'width:100%;')) }}
\ No newline at end of file diff --git a/resources/views/modals/partials/manufacturer-select.blade.php b/resources/views/modals/partials/manufacturer-select.blade.php index 3472e5984a..810292067e 100644 --- a/resources/views/modals/partials/manufacturer-select.blade.php +++ b/resources/views/modals/partials/manufacturer-select.blade.php @@ -1,7 +1,6 @@
-
+
diff --git a/resources/views/modals/statuslabel.blade.php b/resources/views/modals/statuslabel.blade.php index 7e07854197..fba829ae50 100644 --- a/resources/views/modals/statuslabel.blade.php +++ b/resources/views/modals/statuslabel.blade.php @@ -10,21 +10,18 @@
-
-
+ @include('partials.forms.edit.name', ['item' => new \App\Models\Statuslabel(),'translated_name' => trans('admin/statuslabels/table.name') ])
-
-
diff --git a/resources/views/modals/supplier.blade.php b/resources/views/modals/supplier.blade.php index 0dd10595bf..806b57c1de 100644 --- a/resources/views/modals/supplier.blade.php +++ b/resources/views/modals/supplier.blade.php @@ -10,15 +10,12 @@
-
-
+ @include('partials.forms.edit.name', [ 'item' => new \App\Models\Supplier(), 'translated_name' => trans('admin/suppliers/table.name')])
-
diff --git a/resources/views/partials/forms/edit/name.blade.php b/resources/views/partials/forms/edit/name.blade.php index f4d29e606c..5a083bf928 100644 --- a/resources/views/partials/forms/edit/name.blade.php +++ b/resources/views/partials/forms/edit/name.blade.php @@ -1,8 +1,8 @@
-
- +
+ {!! $errors->first('name', '') !!}
From ef82f954e538d8afb6e765aadcffe79b914c276a Mon Sep 17 00:00:00 2001 From: Godfrey M Date: Wed, 25 Sep 2024 15:47:57 -0700 Subject: [PATCH 7/7] fixed required css, and user input lengths --- resources/views/modals/location.blade.php | 2 +- resources/views/modals/model.blade.php | 4 ++-- .../modals/partials/categories-select.blade.php | 5 ++++- resources/views/modals/partials/name.blade.php | 5 ++++- resources/views/modals/statuslabel.blade.php | 4 ++-- resources/views/modals/user.blade.php | 14 +++++++------- .../partials/forms/edit/company-select.blade.php | 2 +- .../forms/edit/location-profile-select.blade.php | 2 +- .../views/partials/forms/edit/name-first.blade.php | 3 ++- .../views/partials/forms/edit/name-last.blade.php | 1 + .../views/partials/forms/edit/username.blade.php | 5 ++++- 11 files changed, 29 insertions(+), 18 deletions(-) diff --git a/resources/views/modals/location.blade.php b/resources/views/modals/location.blade.php index 9817e1ddd7..9580f4bbf9 100644 --- a/resources/views/modals/location.blade.php +++ b/resources/views/modals/location.blade.php @@ -9,7 +9,7 @@
- @include('modals.partials.name') + @include('modals.partials.name', ['item' => new \App\Models\Location(), 'required' => 'true'])
diff --git a/resources/views/modals/model.blade.php b/resources/views/modals/model.blade.php index 3b6bd47e5f..fb1650eee7 100644 --- a/resources/views/modals/model.blade.php +++ b/resources/views/modals/model.blade.php @@ -9,8 +9,8 @@ - @include('modals.partials.name') - @include('modals.partials.categories-select') + @include('modals.partials.name', ['required' => 'true']) + @include('modals.partials.categories-select', ['required' => 'true']) @include('modals.partials.manufacturer-select') @include('modals.partials.model-number') @include('modals.partials.fieldset-select') diff --git a/resources/views/modals/partials/categories-select.blade.php b/resources/views/modals/partials/categories-select.blade.php index 9cbc721748..116a0808e9 100644 --- a/resources/views/modals/partials/categories-select.blade.php +++ b/resources/views/modals/partials/categories-select.blade.php @@ -1,8 +1,11 @@ +@php + $required = $required ?? ''; +@endphp
- +
\ No newline at end of file diff --git a/resources/views/modals/partials/name.blade.php b/resources/views/modals/partials/name.blade.php index 92bd7a62b7..6a64ed525f 100644 --- a/resources/views/modals/partials/name.blade.php +++ b/resources/views/modals/partials/name.blade.php @@ -1,7 +1,10 @@ +@php + $required = $required ?? ''; +@endphp
-
+
\ No newline at end of file diff --git a/resources/views/modals/statuslabel.blade.php b/resources/views/modals/statuslabel.blade.php index fba829ae50..e7076f1bbb 100644 --- a/resources/views/modals/statuslabel.blade.php +++ b/resources/views/modals/statuslabel.blade.php @@ -10,13 +10,13 @@
- @include('partials.forms.edit.name', ['item' => new \App\Models\Statuslabel(),'translated_name' => trans('admin/statuslabels/table.name') ]) + @include('partials.forms.edit.name', ['required' => 'true', 'item' => new \App\Models\Statuslabel(),'translated_name' => trans('admin/statuslabels/table.name') ])
-
{{ Form::select('type', $statuslabel_types, '', array('class'=>'select2', 'style'=>'width:100%','id' =>'modal-type')) }}
+
{{ Form::select('type', $statuslabel_types, '', array('class'=>'select2', 'style'=>'width:100%','id' =>'modal-type', 'required',)) }}
diff --git a/resources/views/modals/user.blade.php b/resources/views/modals/user.blade.php index f04689569b..5672119fa1 100644 --- a/resources/views/modals/user.blade.php +++ b/resources/views/modals/user.blade.php @@ -42,16 +42,16 @@ @include ('partials.forms.edit.location-profile-select', ['translated_name' => trans('general.location'), 'fieldname' => 'location_id'])
- @include('partials.forms.edit.name-first', ['class' => 'col-md-8 col-xs-12-pull', 'style' => 'width:67%;']) + @include('partials.forms.edit.name-first', [ 'required' => 'true', 'class' => 'col-md-8 col-xs-12-pull', 'style' => 'width:67%;'])
- @include('partials.forms.edit.name-last', ['class' => 'col-md-8 col-xs-12-pull', 'style' => 'width:67%;']) + @include('partials.forms.edit.name-last', ['required' => 'true', 'class' => 'col-md-8 col-xs-12-pull', 'style' => 'width:67%;'])
@include('partials.forms.edit.email')
- @include('partials.forms.edit.username') + @include('partials.forms.edit.username', [ 'required' => 'true'])
@@ -59,8 +59,8 @@
-
- + @@ -72,8 +72,8 @@
-
- +
+
diff --git a/resources/views/partials/forms/edit/company-select.blade.php b/resources/views/partials/forms/edit/company-select.blade.php index 012e032866..59de71a6ab 100644 --- a/resources/views/partials/forms/edit/company-select.blade.php +++ b/resources/views/partials/forms/edit/company-select.blade.php @@ -20,7 +20,7 @@
{{ Form::label($fieldname, $translated_name, array('class' => 'col-md-3 control-label')) }} -
+
@if ($location_id = old('location_id', (isset($user)) ? $user->location_id : ''))