Skip to content

menu: Write tests on templates#791

Merged
gela-d merged 1 commit intoissues/753@v2from
issues/779@v2
Jul 23, 2014
Merged

menu: Write tests on templates#791
gela-d merged 1 commit intoissues/753@v2from
issues/779@v2

Conversation

@lapinlapin
Copy link

Close #779

@lapinlapin lapinlapin mentioned this pull request Jul 8, 2014
@tadatuta tadatuta added this to the v2.0 milestone Jul 9, 2014
@lapinlapin
Copy link
Author

@sipayRT

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

добавить проверку на эскейпинг

@sipayRT
Copy link
Contributor

sipayRT commented Jul 14, 2014

добавить тест на mods : { type : 'link' }

@lapinlapin
Copy link
Author

@sipayRT

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

можно удалить

@lapinlapin
Copy link
Author

@sipayRT

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

удали theme

@sipayRT
Copy link
Contributor

sipayRT commented Jul 14, 2014

ok

@gela-d
Copy link
Member

gela-d commented Jul 18, 2014

Надо сделать два примера:
1 пример - меню, в котором итемы незадизейбленные, у меню модификатор disabled + theme
2 пример - меню с группой, у которой есть title, в которой итемы 1 обычный и 1 неактивный с типом ссылки, + в этом примере у меню модификатор focused

gela-d pushed a commit that referenced this pull request Jul 23, 2014
menu: Write tests on templates
@gela-d gela-d merged commit b7fcdc3 into issues/753@v2 Jul 23, 2014
@gela-d gela-d deleted the issues/779@v2 branch July 23, 2014 12:18
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.

4 participants