Customizing bootstrap #350
Conversation
| @import "bootstrap/modals"; | ||
| @import "bootstrap/tooltip"; | ||
| // @import "bootstrap/popovers"; | ||
| // @import "bootstrap/carousel"; |
There was a problem hiding this comment.
We can simply remove the commented out ones, right?
There was a problem hiding this comment.
I left those comments because it could help future contributors understand that we are not loading all the modules. It's kind of an explicit way of saying which Bootstrap parts are not being used.
There was a problem hiding this comment.
IMO that could be a bit more clear if precede it with a comment block explaining just that.
mllocs
force-pushed
the
feature/custom-bootstrap
branch
from
a12e781 to
8181170
Compare
mllocs
force-pushed
the
feature/custom-bootstrap
branch
from
8181170 to
c10b3fb
Compare
| <%= render 'application/menus/inquiries_list_link' %> | ||
| <%= render 'application/menus/offers_by_tag_link' %> | ||
| <% if current_user.manages? current_organization %> | ||
| <%= render 'application/menus/offers_by_tag_link' %> |
There was a problem hiding this comment.
is this now only for admins?
There was a problem hiding this comment.
ping @mllocs
cc/ @sseerrggii
There was a problem hiding this comment.
During the last redesign conversations we had it was proposed to remove this section (to simplify the interface). Instead, I'm making it available only for admins.
3 participants
Uh oh!
There was an error while loading. Please reload this page.