//remove old list, if any
menu.all('ul').remove();
.
.
.
//create new <ul><li></li>...<ul> list
var list = Templates.overviewRelationMenuList({
endPoints: endpoints
});
list.all('li').on('click', ...
menu.append(list);
.
.
.
This way you remove some html elements from this js file. wdyt?
https:/ /codereview. appspot. com/6736051/ diff/1/ app/views/ environment. js environment. js (right):
File app/views/
https:/ /codereview. appspot. com/6736051/ diff/1/ app/views/ environment. js#newcode1449 environment. js:1449: var menu = one('#ambiguous -relation- menu'),
app/views/
container.
If you are using ids you could simply use Y.one, right? Or you could use
a class instead.
https:/ /codereview. appspot. com/6736051/ diff/1/ app/views/ environment. js#newcode1463 environment. js:1463: m.id].forEach( function( endpoint) {
app/views/
endpoints[
You could use handlebars to build the list for you. For example:
//remove old list, if any 'ul').remove( ); </li>.. .<ul> list overviewRelatio nMenuList( {
menu.all(
.
.
.
//create new <ul><li>
var list = Templates.
endPoints: endpoints
});
list.all( 'li').on( 'click' , ...
menu.append(list);
.
.
.
This way you remove some html elements from this js file. wdyt?
https:/ /codereview. appspot. com/6736051/ diff/1/ lib/views/ stylesheet. less stylesheet. less (right):
File lib/views/
https:/ /codereview. appspot. com/6736051/ diff/1/ lib/views/ stylesheet. less#newcode145 stylesheet. less:145: #service-menu, #ambiguous- relation- menu {
lib/views/
You could remove the id and use both classes in your menu. Like...
<div class="service-menu ambiguous- relation- menu"> triangle" > </div>
<div class="
<ul/>
</div>
</div>
So you can remove this new css declaration and do something like...
var menu = container. one('.ambiguous -relation- menu')
...in environment.js
https:/ /codereview. appspot. com/6736051/