change laravel blade css class by the amount of objects

Question

first of all, the question is not about the solution, but about the best practice.

my blade is:

@if (!empty($amount))
     <div class="amount {{$color}}">
          {{$amount}}
     </div>
@endif

color class is defined in service by amount. But I can bet, that is bad architectural solution.

So I need an advice for the best practice in laravel for customizing blade class.

this is bad? im using this technique. it's interesting if anyone can make a better suggestions. — kapitan
– kapitan, Commented Dec 18, 2018 at 6:06
thanks for that Martynas, but im still just using it like so: @ php $bgcolor = {{ $record->amount ? 'success' : 'danger' }} @ endphp then <div class="bg-{{$bgcolor}}"></div> . so this will suffice for now since it is not that complicated. i think that you shouldnt feel guilty using this technique if the logic was very simple (wink) — kapitan
– kapitan, Commented Dec 18, 2018 at 6:42

Mario Haubenwallner · Accepted Answer · 2018-12-18 09:38:26Z

1

I would calculate the color level ($color in your example) where you assign $amount to the blade template (most likely in a controller). You can do this with if/elseif/else where you set colors depending on your limits (e.g. $amount < 5 is green, $amount < 10 is orange, $amount < 20 is red).

It is best practice to separate design from logic, therefore I won't recommend to add logic to you blade template - even if it is a one-liner.

If that's the way you have it right now then I'd keep it this way.

answered Dec 18, 2018 at 9:38

Mario Haubenwallner

1,9651 gold badge19 silver badges22 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

Martynas Tumas Over a year ago

data is assigned in view composer. So I added additional parameter there. But I have another tricky question - where I should put method to return that class name by amount (e.g. $amount < 5 is green, $amount < 10 is orange, $amount < 20 is red) ?

Mario Haubenwallner Over a year ago

I'm a big fan of the "Repository Design Pattern". For example I have a UserController and respectively a UserRepository. So my controller has a function call to the UserRepository ($userRepository->calculateSomething()) and nothing more. I have almost no logic in my controllers - everything is in my repositories.

kapitan Over a year ago

but what if you're inside the foreach loop on the blade file?

Mario Haubenwallner Over a year ago

In the blade foreach-loop you are outputting data for each element of a (hopefully previously prepared and to the view assigned) array/object. It doesn't matter if you have one or many colors. It only depends on your data structure.

Collectives™ on Stack Overflow

change laravel blade css class by the amount of objects

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related