issues
search
jeroennoten
/
Laravel-AdminLTE
Easy AdminLTE integration with Laravel
MIT License
3.82k
stars
1.08k
forks
source link
Improve the Lang menu filter (stage 2)
#1269
Closed
dfsmania
closed
5 months ago
dfsmania
commented
5 months ago
Question
Answer
Pull request type
ENHANCEMENT
License
MIT
What's in this PR?
Improves the
LangFilter
of the menu:
Simplify the filter constructor by eliminating the Translator instance, now we use the
Lang Facade
instead.
Better code documentation on the filter.
Split tests related to this filter on a separated class.
Checklist
[x] I tested these changes.
What's in this PR?
Improves the
LangFilter
of the menu:Checklist