Skip to content
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

Adding updates to tweak the dashboard. #41

Merged
merged 2 commits into from
Feb 16, 2024
Merged

Adding updates to tweak the dashboard. #41

merged 2 commits into from
Feb 16, 2024

Conversation

tnylea
Copy link
Contributor

@tnylea tnylea commented Feb 16, 2024

While working on a way to add multiple dashboard shells which include styles that have a sidebar and other variations of dashboard templates. These tweaks were needed to make all the pages consistent.

Copy link

what-the-diff bot commented Feb 16, 2024

PR Summary

  • Enhanced Aesthetic Value and Space Management in Class Layout
    Aesthetic values and space management were improved in the class layout by adding a mb-5 class to the header element in app.blade.php. This increases the margin at the bottom of the header, providing better space distribution.

  • Updated Class Structure in Dashboard Page
    The structure of classes in the dashboard page was updated to optimize the layout in index.blade.php, leading to a cleaner and more efficient code.

  • Applied Consistent Styling Across Layouts
    The same mb-5 class was added to the header element in the app.blade.php of default layout. This ensures consistency across different layouts and maintains a uniform style.

  • Cleanup in Dashboard Code
    Removed irrelevant and unused code comments in the dashboard page index.blade.php under default resources, decluttering the code and making it easier to understand and maintain.

  • Removed Unnecessary Code
    Erased the last line of the code in index.blade.php under the dashboard page of the default resources. This was an extra line that served no functionality and could cause confusion during code interpretation.

@tnylea tnylea merged commit 1b37fac into main Feb 16, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant