From 3509098bbc87440ed8b314f5e432f85ea14fa60f Mon Sep 17 00:00:00 2001
From: ttimot24 <ttimotster@gmail.com>
Date: Thu, 14 Nov 2024 14:34:13 +0100
Subject: [PATCH] Improved user feedback on blogpost forms

---
 app/Libs/PluginManager.php        |  7 ++++++-
 app/Model/Blogpost.php            |  1 +
 app/View/blogposts/form.blade.php | 34 +++++++++++++++++++++----------
 app/View/users/form.blade.php     | 10 +++++++--
 4 files changed, 38 insertions(+), 14 deletions(-)

diff --git a/app/Libs/PluginManager.php b/app/Libs/PluginManager.php
index 7e44dc7ed..50baf737e 100644
--- a/app/Libs/PluginManager.php
+++ b/app/Libs/PluginManager.php
@@ -2,9 +2,14 @@
 
 namespace App\Libs;
 
+/**
+ * @deprecated deprecated since version 1.0.0
+ */
 class PluginManager extends Model{
 
-
+	/**
+	 * @deprecated deprecated since version 1.0.0
+	 */
 	public static function area($area_num){
 
 		foreach(app()->plugins as $widget){
diff --git a/app/Model/Blogpost.php b/app/Model/Blogpost.php
index 6f582a5bf..31c77194e 100644
--- a/app/Model/Blogpost.php
+++ b/app/Model/Blogpost.php
@@ -29,6 +29,7 @@ class Blogpost extends Model
 
     public static $rules = [
         'title' => 'required',
+        'summary' => 'max:255',
         'category_id' => 'required'
     ];
 
diff --git a/app/View/blogposts/form.blade.php b/app/View/blogposts/form.blade.php
index 2536c9a77..c02bb2bdf 100644
--- a/app/View/blogposts/form.blade.php
+++ b/app/View/blogposts/form.blade.php
@@ -21,14 +21,20 @@
 
                         <div class="col-xs-12 col-md-8">
 
-                            <div class='form-group pull-left col-12'>
-                                <label for='title'>{{ trans('blogpost.title') }}:</label>
-                                <input type='text' class='form-control' id='title' name='title'
+                            <div class='form-group col mb-4'>
+                                <label for='title'>{{ trans('blogpost.title') }}</label>
+                                <input type='text' class='form-control @error('title') is-invalid @enderror' id='title' name='title'
                                     value="{{ old('title', isset($blogpost) ? $blogpost->title : '') }}" required>
+
+                                @error('title')
+                                    <span class="invalid-feedback" role="alert">
+                                        <strong>{{ $errors->first('title') }}</strong>
+                                    </span>
+                                @enderror
                             </div>
 
-                            <div class='form-group pull-left col-6'>
-                                <label for='sel1'>{{ trans('blogpost.select_category') }}:</label>
+                            <div class='form-group col-6 mb-4'>
+                                <label for='sel1'>{{ trans('blogpost.select_category') }}</label>
                                 <select class='form-select' name='category_id' id='sel1'>
 
                                     @foreach ($categories as $category)
@@ -40,10 +46,16 @@
                                 </select>
                             </div>
 
-                            <div class='form-group pull-left col-12'>
-                                <label for='title'>{{ trans('blogpost.summary') }}:</label>
-                                <input type='text' class='form-control' id='title' name='summary'
-                                    value="{{ old('summary', isset($blogpost) ? $blogpost->summary : '') }}"></br>
+                            <div class='form-group col  mb-4'>
+                                <label for='title'>{{ trans('blogpost.summary') }}</label>
+                                <textarea type='text' class='form-control @error('summary') is-invalid @enderror' id='summary' name='summary'>{{ old('summary', isset($blogpost) ? $blogpost->summary : '') }}</textarea>
+                                </br>
+
+                                @error('summary')
+                                    <span class="invalid-feedback" role="alert">
+                                        <strong>{{ $errors->first('summary') }}</strong>
+                                    </span>
+                                @enderror
                             </div>
 
                         </div>
@@ -57,7 +69,7 @@
                             @endif
 
                             <div class='form-group'>
-                                <label for='file'>{{ trans('actions.upload_image') }}:</label>
+                                <label for='file'>{{ trans('actions.upload_image') }}</label>
                                 <input name='up_file' accept='image/*' id='input-2' type='file' class='file'
                                     multiple='true'
                                     data-drop-zone-enabled="{{ isset($blogpost) && $blogpost->hasImage() ? 'false' : 'true' }}"
@@ -68,7 +80,7 @@
 
                         <div class="col-12">
                             <div class='form-group pull-left col-12'>
-                                <label for='text'>{{ trans('blogpost.post') }}:</label>
+                                <label for='text'>{{ trans('blogpost.post') }}</label>
                                 <text-editor id="texteditor" :name="'text'"
                                     :data="'{{ remove_linebreaks(old('blogpost', isset($blogpost) ? $blogpost->text : '')) }}'"
                                     :language="'{{ config('app.locale') }}'"
diff --git a/app/View/users/form.blade.php b/app/View/users/form.blade.php
index 0d9d9cdfd..1ecb0e95c 100644
--- a/app/View/users/form.blade.php
+++ b/app/View/users/form.blade.php
@@ -23,7 +23,7 @@
                         <div class='form-group pull-left col-xs-12 col-md-8'>
                             <div class='form-group'>
                                 <label for='name'>{{ trans('user.create_name') }}:</label>
-                                <input type='text' class='form-control' id='name' name='name'
+                                <input type='text' class='form-control @error('name') is-invalid @enderror' id='name' name='name'
                                     value="{{ old('name', isset($user) ? $user->name : '') }}" required autofocus>
                             </div>
 
@@ -61,8 +61,14 @@ class="form-control @error('password') is-invalid @enderror"
 
                             <div class='form-group'>
                                 <label for='title'>{{ trans('user.create_email') }}:</label>
-                                <input type='email' class='form-control' id='title' name='email'
+                                <input type='email' class='form-control  @error('email') is-invalid @enderror' id='email' name='email'
                                     value="{{ old('email', isset($user) ? $user->email : '') }}" required>
+                                
+                                @error('email')
+                                    <span class="invalid-feedback" role="alert">
+                                        <strong>{{ $errors->first('email') }}</strong>
+                                    </span>
+                                @enderror
                             </div>
 
                             <div class='form-group'>