Rails - Dec 2006 - How to associate a model's attribute with CSS style

So this isn''t a question specific to Rails, but I wanted to keep the
purity of the MVC architecture as much as possible.

I have an AR model, order, which can have a status.  I want to style
the orders based on what their status is.  Right now all my status AR
model has is a name and I''m creating the CSS class in my view with
something like:

class="<%= order.status.name %>"

but I don''t like how coupled this makes everything.  I''m sure
people
have figured out a best practice for this.  I was thinking maybe adding
a style attribute to Status?  Anyone else have a better suggestion?

Thanks,
Joe


--~--~---------~--~----~------------~-------~--~----~
 You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

I don''t see how this is incorrectly coupled, you are printing out the
status name in the view which the view then styles accordingly. Adding
a style attribute to the model would be incorrect because then the
model would know about the view.

Since the styling only makes sense in this particular html view (if
you were to render as xml it would be unnecessary) then that seems
like the best place to me.

On 12/31/06, jjathman-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org
<jjathman-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
wrote:
>
> So this isn''t a question specific to Rails, but I wanted to keep
the
> purity of the MVC architecture as much as possible.
>
> I have an AR model, order, which can have a status.  I want to style
> the orders based on what their status is.  Right now all my status AR
> model has is a name and I''m creating the CSS class in my view with
> something like:
>
> class="<%= order.status.name %>"
>
> but I don''t like how coupled this makes everything.  I''m
sure people
> have figured out a best practice for this.  I was thinking maybe adding
> a style attribute to Status?  Anyone else have a better suggestion?
>
> Thanks,
> Joe
>
>
> >
>

-- 
Ross Riley
www.sorrylies.com

--~--~---------~--~----~------------~-------~--~----~
 You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

I don''t see this as a violation of MVC at all.  In fact, I''d
say that
adding a style attribute to status *would* be a violation.

One thing i would say is that it''s probably worth making a helper to
get the css class:

def css_style_from(text)
   text.tablelize
end

class="<%= css_style_from(order.status.name) %>"

or something like that.  Tablelizing the text will mean you don''t get
problems with spaces in status names:
Order Complete becomes order_complete

It''s not great to use tablelize in the view as you may change your CSS
naming rules, so a nice little helper will keep you DRY.

Hope that helps,

Steve
www.curve21.com


--~--~---------~--~----~------------~-------~--~----~
 You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Yeah, I figured out early on that having spaces in a style is not
possible.  The only thing I''m worried about is that if I make a style
for "pending" it will be a programming change (changing the
style''s
name) if the user''s ever want to change what the name is,
"unprocessed"
for example.  Not a big deal, but I never like having to make
programming changes when an arbitrary value is changed.  Thoughts?
Thanks for the help!

Joe


--~--~---------~--~----~------------~-------~--~----~
 You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

> Not a big deal, but I never like having to make
> programming changes when an arbitrary value is changed.  Thoughts?
I''d say that unless the client *needs* to update the status name
regularly, it''s unnecessary to provide them with that functionality.  I
guess it comes down to the system in the end.  If this is a system
where the styles and names are changed regularly, then storing styling
in the db might not be such a bad thing.  However, in most cases it''s
probably not needed and you should choose the method that makes your
life easier and benefits the application. 

Steve


--~--~---------~--~----~------------~-------~--~----~
 You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---