reed-college / strabo

Interactive mapping site with database
0 stars 1 forks source link

Interest point edit table #30

Closed benblack769 closed 8 years ago

benblack769 commented 8 years ago

In order to make the interest point edit table more clear and more descriptive. I think we should make the following changes to the table:

  1. Change the layer number to layer name.
  2. Add icon image.
  3. Add thumbnails of the images to the back of the table.

So the table would look like this:

Edit Delete Title Description Layer Icon Images
Edit Delete Fish Ladder Reed Lake was created sometime near the turn of the century by the construction of a 10-foot-high dam across Reed Creek. The creation of the dam blocked fish passage, as the creek was re-routed through a culvert system that discharged to a steep waterfall. \n \n Over the summer and fall of 2001, Reed College constructed a fish ladder to re-establish connectivity between Reed Lake and the lower creek for resident and anadromous fish. Interest Points alt text alt text alt text

Except the images would be lined up horizontonally, instead of vertically.

rekennelly commented 8 years ago

I think the images being lined up vertically could be good in the case of a long description like above. What do you think the benefit of horizontally lining up the images would be?

benblack769 commented 8 years ago

I mean, on a full size desktop screen at our current font size, the height of even that long description will only be the height of one of those images. So if our images were vertically aligned, then it would be taking up unnecessary screen space. On a smaller screen, I suspect an html table would make the images align vertically by default anyways. I would have to check that, though.

kamba16 commented 8 years ago

where do we stand on this? should I work on this using the current admin interface in master? should I put this off until we figure out what we want to do with admin_redo?

benblack769 commented 8 years ago

This shouldn't change too much between admin_redo and the current master. So go ahead.