-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Configure the content-header of template #918
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a fallback, if the config var is not set.
With fallback
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add fallback
@warquia what classes did you have added to the new config option you have created to get from picture 1 to picture 2. If the goal is to not shown the {{-- Content Header --}}
@hasSection('content-header')
<div class="content-header">
<div class="{{ config('adminlte.classes_content_header') ?: $def_container_class }}">
@yield('content_header')
</div>
</div>
@endif Also, note when you do: <div class="{{ config('adminlte.classes_content_header_main' , 'content-header') }}"> the default value will only be used when the configuration option do not exists. So, when defining the option to <div class="{{ config('adminlte.classes_content_header_main') ?: 'content-header' }}"> So, when Also, next time avoid using the |
Good morning everyone, really has-section is the best approach, I will correct the pull-request with your analysis. Thank you all |
a better approach was taken
A better approach was taken
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, do not delete config/adminlte.php
file. Just use git checkout config/adminlte.php
to restore it to the original one, then commit that one. That way you will rollback the first modifications you have done to that file.
@resslinger @warquia This PR is ok for me now... |
| Question | Answer
| ----------------------- | [-----------------------]
| Issue or Enhancement | Enhancement
| License | MIT
What's in this PR?
Allowing you to generate or not a content-header, thus allowing at 1024x768 resolutions gains more screen space,
Checklist