openwisp / django-ipam

The development of this project has moved to openwisp-ipam
https://github.com/openwisp/openwisp-ipam
BSD 3-Clause "New" or "Revised" License
78 stars 28 forks source link

[subnet-visualizer] Invalid HTML #96

Closed nemesifier closed 4 years ago

nemesifier commented 4 years ago

In the infinite scrolling subnet visualizer, I noticed that a <div> is used to load pages, but that <div> is inside a <ul> element, which makes it not valid HTML. We should change the structure of the HTML so that it's valid.

We can use something like <section> > <div> > <a> (instead of <ul> > <div> > <li><a>.

CC: @pawelplsi

nepython commented 4 years ago

@nemesisdesign , I checked the templates but found the <ul> within <div> and not viceversa at changeform.html. It maybe that I am viewing the wrong template, if so can you please tell me the correct one :)

pawelplsi commented 4 years ago

@nepython It's not in a template, these divs are added dynamicaly by a script. Anyway, I am working on solving it right now.