gaiaresources / ala-citizenscience

Automatically exported from code.google.com/p/ala-citizenscience
1 stars 4 forks source link

The census method attribute table can overflow the page in themes #309

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Create a new theme and give it a content width less than the default.
2. Go to the Admin -> Manage Projects -> Manage Census Methods
3. Pick a census method if you have one or create a new one
4. Add a row to the attribute table and view that it extends beyond the bounds 
of the container.

Original issue reported on code.google.com by stepha...@gaiaresources.com.au on 17 Jul 2012 at 8:46

GoogleCodeExporter commented 9 years ago
This is a common problem for attribute tables. I propose to fix it by enclosing 
it in a scrollable div, keeping the entire table inside the input container, 
but allowing the user to scroll to modify the right side of the table. The 
table already has an Enlarge/Shrink table link so it could always be edited as 
a full screen table if the scrolling was too inconvenient.

Original comment by stepha...@gaiaresources.com.au on 17 Jul 2012 at 8:46

GoogleCodeExporter commented 9 years ago
Design review comments by kehan:

Sounds good but watch out for IE7, and obviously only make it scrollable on the 
x axis.

Original comment by stepha...@gaiaresources.com.au on 17 Jul 2012 at 8:47

GoogleCodeExporter commented 9 years ago
Added the "scrollable" class which makes a div scrollable along the x axis.

Original comment by stepha...@gaiaresources.com.au on 17 Jul 2012 at 8:47

GoogleCodeExporter commented 9 years ago
Fixed in r377.

Original comment by stepha...@gaiaresources.com.au on 25 Jul 2012 at 1:48