Details
Bug
Major
Description
If the content of the accordion exceeds it's parent container height, the content of the opened pane does not scroll. If we set overflow: auto in css on that element, it scrolls the whole accordionPane, not just the content due to how the elements are organized:
<div>
---<section>
---------<div class="handle">...</div>
--------- content
-- </section>
</div>
Activity
If the contents of the accordion pane do not scroll, and we allow fixed height on the pane, then the contents may become hidden. This isn't good.
Ted, could you please quickly comment on why we shouldn't allow scrolling in the accordion panes? thx
My comment was mostly about the UI design – requiring the user to open several tiny panels and then scroll within them seems to defeat the purpose of the accordion. They have selected the category of interest so should be able to easily select from that category.
As far as prohibiting it within the component, that's not necessary – you can imagine an accordion with a very large number of items in each category. The large number of items would also indicate a UI problem.
So, a scrollable attribute sounds fine, but I doubt we should use it in our demos.
If fixedHeight is specified and the contents are larger than the fixedHeight they will now scroll
| Field | Original Value | New Value |
|---|---|---|
| Reporter | Migration [ remote ] | User Philip.breau [ philip.breau ] |
| Assignee | Steve Maryka [ steve.maryka ] | Ted Goddard [ ted.goddard ] |
| Status | Open [ 1 ] | Resolved [ 5 ] |
| Resolution | Fixed [ 1 ] |
| Resolution | Fixed [ 1 ] | |
| Status | Resolved [ 5 ] | Reopened [ 4 ] |
| Assignee | Ted Goddard [ ted.goddard ] | Judy Guglielmin [ judy.guglielmin ] |
| Fix Version/s | 1.2 Final [ 10371 ] |
see MOBI-533
| Status | Reopened [ 4 ] | Resolved [ 5 ] |
| Resolution | Duplicate [ 3 ] |
when I asked Ted if this should be an attribute for this component, he stated that the contents of an accordion should never scroll. If it's a desirable feature, then it should be an attribute (so that those who don't want the scrolling don't have to have it), perhaps?