Как задать класс для определенных страниц?

Question

[lavezzi1]

Здравствуйте. Есть такой код:

В application.html.haml

- if user_signed_in?
			%nav.navigation
				%ul
					%li
						= link_to "New project", new_place_path
					%li
						= link_to "Profile"
					%li
						= link_to "Sign Out", destroy_user_session_path, method: :delete

		- else
			%nav.navigation_welcome
				.inner
					%ul
						%li
							= link_to "Login", new_user_session_path
						%li
							= link_to "Sign up", new_user_registration_path

На главной странице фон менюшки сверху прозрачный и при переходе на страницы типа /login & /sign_up нужно поменять цвет меню. Как вообще лучше это реализовать?

Answer 1

[ТыжСисАдмин]

$(function () {
    var url = window.location;
    $('#top-menu a[href="' + url + '"]').parent('li').addClass('current-page');
    $('#top-menu a').filter(function () {
        return this.href == url;
    }).parent('li').addClass('current-page').parent('ul').slideDown().parent().addClass('active');
});

<div id="top-menu">
 <ul class="nav child_menu">
  <li><a href="index.html">Ololo</a></li>
  <li><a href="index2.html">Ololo2</a></li>
 </ul>
</div>

Добавляет класс current-page к пункту меню, href которого совпадает с текущим URL.
domen.com/index2.html

<li class="current-page"><a href="index2.html">Ololo2</a></li>

Comments

[Павел Кононенко]

Это же лютый говнокод

[ТыжСисАдмин]

Павел Кононенко: согласен, но такое юзается сплошь и рядом, в том числе и в некоторых крупных проектах.
Скажем так, это не говнокод а велосипед для мелких менюх, трёхколёсный, с кривым рулём :)

[CapeRatel]

Это рабочее решение, для не страшных менюх. Нормально все. Хотя можно пильнуть наждачкой по js.

Answer 2

[Павел Кононенко]

Можно написать свой хэлпер, можно сделать before_action в вашем HomeController, который будет взводить переменную.
можно и в лоб

nav{ class: "#{'transparent' if params[:controller]!='home'}" }

Можно привязываться не к контроллеру, а к request.url

Самый лучший способ – использовать ангуляр)

Comments

[lavezzi1]

А вообще правильно что я написал меню для всех страниц в application.html.haml?
Или может для каждый страницы свое меню выводить...ума не приложу как сделать лучше и правильней

[Павел Кононенко]

lavezzi1: навигация должна быть в лейауте. Это правильный путь. Ну можно конечно сделать content_for, но по-моему это будет извратом... Сам не знаю более красивого решения...

[Viktor Vsk]

lavezzi1: вопрос сильно ни о чем. для кого правильнее, в каких ситуациях - не понятно
но с другой стороны, логичным кажется, что если у вас в зависимости от роли юзера (гость, админ, пользователь) меняются пункты меню, можно их явно каждый проверять (if current_user.try(:admin?) , if current_user.try(:customer?) и не перерендеривать заново весь блок меню. А если различия в разметке значительные (например, у вас уже идет структура меню не ul -> li, а ul -> .inner -> li -- то можно подумать и о разных лейаутах: для гостя, пользователя, админа.

Сами просто представьте, что если нужно будет для админа рендерить расширенное-мигающее-меню - обернете все в еще один иф?

[lavezzi1]

Виктор Ablebeam: Нет никаких админов. Есть гость и пользователь, причем главную страницу видит только гость, пользователь же работает только с dashboard и на главную перейти не может. Суть вопроса в том, как спрятать или изменить стилистически меню которое идет после else на странице регистрация и вход

[Viktor Vsk]

lavezzi1: суть в том, что вы никогда не знаете, какой код придется писать через месяц
Стилистически все делается с помощью CSS, то есть, решается добавлением определенного css-класса.