ixmaps / website2017

Website for the IXmaps project
https://www.ixmaps.ca
GNU Affero General Public License v3.0
4 stars 2 forks source link

Map page issues #48

Open Andrew-Clement opened 7 years ago

Andrew-Clement commented 7 years ago

Various refinements, mainly in appearance tot he Map page

Do the easy ones first. Note in a comment any that are time-consuming to be done later

Opening modal

Appear on default map background, not black

See Issue #47 for the default map

Skip modal, if…

Closing the modal (X) should turn the background map into a live map the corresponds to the default background map.

NOTE: All modals should appear on previous screen, not with black backgrounds

Traceroutes

Currently shows a sample of routes, not most recent 20. Displaying 1 of 20 most recent search results Preferably list the most recent 20, or if this is difficult, reword to Displaying 1 of a sample of 20 search results

CLOSE vertical space between Displaying 1 of 20 most recent search results AND
Search results details…

INCREASE very slightly the vertical space between flag/CC and overflow city line

pastedgraphic-7

Carriers

SQUEEZE Routers column, Abbreviate to Rts, centre # of Rts, give more width to Carrier

Incorporate the second vertical scroll in the carrier panel, like that of the Traceroutes panel

pastedgraphic-4

Carrier pop-up

REPLACE To review the full interim report WITH To review the full interim report pastedgraphic-3

TRdetails Avoid duplication of Traceroute details pastedgraphic-1

Below map

REMOVE extraneous buttons at bottom once their testing value is done pastedgraphic-8

Toggle Layer Visibility

Highlight NSA button to correspnd with the display (ie ON by default) pastedgraphic-21

QUICK search

MAKE From My ISP button active

Search query line is nice! Nudge up slightly to appear clser to the button row.

[ Refine in Advanced Search ] should populate the parameters in Advanced Search

It would be nice if this feature could be added to the Basic search as well. (Only do htis in the current round if really easy to do, and after Basic search is functioning better)

pastedgraphic-19

Advanced Search Populate the default attibute options with the first item, not the last, as is currently the case with the Explore page,

ie Does Originate Submitter name AND pastedgraphic-20

Andrew-Clement commented 7 years ago

Additional Map page issues

Opening modal

ENHANCE FUNCTIONALITY

If a search doesn't produce results, OK to show latest route. But this is the only time. After that if nothing is found, simply leave a blank map as repeatedly going to the latest route is confusing

ENHANCE PRESENTATION/INTERACTION Should only appear once in a session.

Settings

ENHANCE PRESENTATION - MINOR Include cog icon to right of Settings. Make font match that of Help link above, but can keep lower case

QUICK search FIX PRESENTATION Search query text for Via NSA City is incorrect. Replace ANDs with ORs pastedgraphic-23

ENHANCE PRESENTATION - MINOR Centre the row of 6 buttons across the full width of the visible box

BASIC search

FIX BUG Search doesn't terminate e.g. Contributor: AndrewC search doesn't terminate

agamba commented 7 years ago

Andrew, please clear history/cache and try the new fixes for the items highlighted in yellow bellow

On Wed, Mar 15, 2017 at 9:16 PM, Andrew-Clement notifications@github.com wrote:

Additional Map page issues

Opening modal

ENHANCE FUNCTIONALITY

If a search doesn't produce results, OK to show latest route. But this is the only time. After that if nothing is found, simply leave a blank map as repeatedly going to the latest route is confusing

I've noticed this too. Showing the last tr is really confusing. I though we can include a button on the pop up message the ask the user whether to load the last tr or not.

ENHANCE PRESENTATION/INTERACTION Should only appear once in a session.

This will probably will require storing some type of cookie on the client

Settings

ENHANCE PRESENTATION - MINOR Include cog icon to right of Settings. Make font match that of Help link above, but can keep lower case

QUICK search FIX PRESENTATION Search query text for Via NSA City is incorrect. Replace ANDs with ORs [image: pastedgraphic-23] https://cloud.githubusercontent.com/assets/12848029/23977419/53e2b3b4-09ab-11e7-874a-87a9ac741a1e.png

BASIC search

FIX BUG Search doesn't terminate e.g. Contributor: AndrewC search doesn't terminate

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ixmaps/website2017/issues/48#issuecomment-286928872, or mute the thread https://github.com/notifications/unsubscribe-auth/AA8_79CWHFXh0WDUEcLa0P72QDILHCTWks5rmI1ggaJpZM4MdEMS .

Andrew-Clement commented 7 years ago

Good.

I suspect opening a cookie would involve some effort, so let's hold off until we deal with beta feedback.

Also, returning visitors to the Map can simply close the modal if they want.