Skip to content

Added $labelOptions for the Nav#128

Open
michaelantipenko wants to merge 1 commit into
yiisoft:masterfrom
michaelantipenko:master
Open

Added $labelOptions for the Nav#128
michaelantipenko wants to merge 1 commit into
yiisoft:masterfrom
michaelantipenko:master

Conversation

@michaelantipenko

Copy link
Copy Markdown
Q A
Is bugfix? no
New feature? yes
Breaks BC? no
Tests pass?
Fixed issues

@michaelantipenko michaelantipenko changed the title Added $labelOptions for the Nav. Added $labelOptions for the Nav Apr 6, 2016
@SilverFire

Copy link
Copy Markdown
Member

What use case does this PR solve?

These are options for <li> tag, not for label. Then we need dropdownCssOptions and linkOptions as properties to stay consistent.

@samdark

samdark commented Apr 6, 2016

Copy link
Copy Markdown
Member

I'd like to see use case as well...

@michaelantipenko

Copy link
Copy Markdown
Author

Sorry, of course <li>. I need set same css class every <li>. I can not set Nav items without prepare or duplicate item options.

@samdark

samdark commented Apr 6, 2016

Copy link
Copy Markdown
Member

What for?

@SilverFire

Copy link
Copy Markdown
Member

Why it can't be solved with class for <ul> and proper CSS?

@michaelantipenko

Copy link
Copy Markdown
Author

Нужно вывести 10 элементов меню, и каждому тегу <li> задать одинаковый класс. Мне нужно сделать цикл и промержить каждый элемент, чтобы не дублировать напрямую в конфигурации виджета. Хотелось иметь возможность задать каждому элементу свойства один раз, чем то похоже как GridView свойства для ячеек таблицы. Можно через css, просто на данный момент пришла верстка с заданным одинаковым для каждого <li> классом.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants