OpenSource-Tools / LiquidVoting

1 stars 0 forks source link

Cleanup Breadcrumb HTML #19

Closed kvdmolen closed 9 years ago

kvdmolen commented 9 years ago

Cleanup Breadcrumb HTML:

Old HTML:

<div class="slot_title" id="slot_title">
    <a title="Home" class="home" href="../../index/index.html"><img src="../../static/icons/48/home.png" class="icon24"></a>
    <span class="spacer"> » </span>
    <span>
        <span class="unit">
            <a href="../../unit/show/2.html"><span class="name">Unit 1</span></a>
        </span>
        <span class="spacer"> » </span>
        <span class="area">
            <a href="../../area/show/3.html"><span class="name">Subject 1</span></a>
        </span>
        <span class="spacer"> » </span>
        <span class="issue">
            <a href="../../issue/show/1.html">Test #1</a><img title="you are interested" class="star" src="../../static/icons/48/eye.png" alt="you are interested">
            <a href="../../delegation/show.html?issue_id=1" class="delegation_info suspended"></a>
        </span>
        <span class="initiative">Test111</span>
    </span>
</div>

New HTML:

<div class="slot_title" id="slot_title">
    <ul>
        <li class="home">
            <a href="/index.html"><i class="fa fa-home"></i></a>
        </li>
        <li class="unit">
            <a href="..."><span>Unit</span></a>
        </li>
        <li class="area [notifications]">
            <a href="..."><span>Subject</span></a>
        </li>
        <li class="issue [notifications] [delegated]">
            <a href="..."><span>Test #1</span></a>
        </li>
        <li class="initiative [notifications] [supporting] last">
            <a href="..."><span>Test111</span></a>
        </li>
    </ul>
</div>

Preferably also add .last to the last <li> element.

nktc commented 9 years ago

This is mostly done: I have restructured the HTML to unordered list, and added classes for notification supporting, interested, and last. Had to move some lua code around for this part.

However, I'm not sure where to find delegated and notifications. Any clarification you can provide would be helpful.

http://178.62.254.91/lf/initiative/show/1.html

kvdmolen commented 9 years ago

NEW THINGS

nktc commented 9 years ago

I got everything here except .subscribed. Will revisit that.

nktc commented 9 years ago

.subscribed done. Sorry, I had to figure out how to get the area delegation info from inside another file.