mezerotm / cmv

CMV is a citySDK based map visualizatoin tool
5 stars 5 forks source link

The table of maps for the 1, 2, and 4 maps needs to be built dynamically #33

Closed hardnett closed 7 years ago

hardnett commented 7 years ago

In working on the titles for the tables. I was testing the idea with 1, 2, and 4 tables. I found out then that the table that holds the maps had some hard-coded styling that made appear to work, but in reality is was not working properly. The problem is that we have a static table that will always have at least 2 rows. So even if the other cells are not showing, the table will just have a row below the single map that is empty and this makes the single map appear to be taking only half of the space. The solution is going to be:

  1. don't have a table for a single map
  2. build a 1 row table for 2 maps
  3. build a 2 row table for 4 maps

This is likely to affect display.maps.js and top.bar.js (where the update takes place).