search

saurabhd / hk_realestate

GNU General Public License v2.0
1 stars 0 forks source link

Map popup max height and future apartment-list solution #37

Open heliogabal opened 7 years ago

heliogabal commented 7 years ago

We now have the case of an object with over 100 apartments inside (K55, nid 3800), so we need to work on the popup display on the map. The list is getting too long to fit on the screen.

As a first emergency measure, it would be enough to make the actual list (table_body) scrollable by changing css to overflow with overflow-y: scroll. That should be done asap to maintain basic usability. map_popup

When showing it to the client then, I would like to be prepared and have some suggestions ready for the client how this could be further improved. That would also concern the object view, where the apartment list is getting confusingly long. So please have a look at this problem and provide an offer for possible solutions.

darshi-shah commented 7 years ago

Hi Rainer,

I have added scroll as you suggested. I am checking if we can use pager in popup. I will let you know if that works and if not, we will suggest something else.

Thanks Darshi

heliogabal commented 7 years ago

Hi Darshi, ok, that's good. I checked and the size should be further restricted to 250px height and the scroll should only occur on the tbody to keep the header visible at all times.

Another question is if it's possible to position the marker with the opened popup not centrally on the map, but with a slight offset (popup upper border ca. 20-30px under the white menu bar). Of course we would have to do something with the search box overlapping the popup then. It's tricky... Like that: popup_position_correct instead of that: popup_under_menu

the client further asked if it is possible to never let the popup slip under the menu, but I don't really see a way to prevent that. But if you have an idea...

darshi-shah commented 7 years ago

Hi Rainer,

Here is what we suggest for more apartments on map:

On map popup, pager and ajax call do not work, we checked this. So instead of showing all apartments on popup itself, we show a button for view more which redirects users to a new page if we have more apartments (let's say more than 5) available.

On this page, we will show all apartments with pager. This page will have all the elements of popup including text above and inquiry button below list.

Please let us know your thoughts on this.

Thanks Darshi

heliogabal commented 7 years ago

Hi Darshi, thanks for the suggestions. I would love to avoid having to guide the visitor away from the map too soon, but probably it is still the best solution. As we were thinking of reworking the somewhat neglected object pages into full blown landing pages, we should leave this on hold until that is finished.

Once we have beautiful object pages that provide search functionality, contact button and the such, we can think about redirecting users to it. So I put this on hold.