rappasoft / laravel-livewire-tables

A dynamic table component for Laravel Livewire
https://rappasoft.com/docs/laravel-livewire-tables/v2/introduction
MIT License
1.71k stars 320 forks source link

v3 Toolbar - Splitting of Views/Blades #1454

Closed lrljoe closed 8 months ago

lrljoe commented 8 months ago

THIS IS IN PROGRESS - DO NOT MERGE OR USE.

This PR separates out the toolbar into blades per function, removes the duplication for themes (except one item), and tries to clean up the approach for maintainability.

All Submissions:

New Feature Submissions:

  1. [x] Does your submission pass tests and did you add any new tests needed for your feature?
  2. [x] Did you update all templates (if applicable)?
  3. [x] Did you add the relevant documentation (if applicable)?
  4. [x] Did you test locally to make sure your feature works as intended?

Changes to Core Features:

lrljoe commented 8 months ago
codecov[bot] commented 8 months ago

Codecov Report

Merging #1454 (e116b8e) into v3-develop (bdb9b8b) will not change coverage. Report is 1 commits behind head on v3-develop. The diff coverage is n/a.

@@              Coverage Diff              @@
##             v3-develop    #1454   +/-   ##
=============================================
  Coverage         88.11%   88.11%           
  Complexity         1222     1222           
=============================================
  Files                85       85           
  Lines              2971     2971           
=============================================
  Hits               2618     2618           
  Misses              353      353           
lrljoe commented 8 months ago

Needs final testing - ensure that no adverse effects