Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A popup on the map. #13

Closed
2 tasks
cchristelis opened this issue Apr 16, 2015 · 3 comments
Closed
2 tasks

A popup on the map. #13

cchristelis opened this issue Apr 16, 2015 · 3 comments
Assignees

Comments

@cchristelis
Copy link
Contributor

A popup on the map should provide the user with information on the selected course/ campus.

Provider popup

This for searches that exclude courses and for the home page where only Providers are mapped by default.

  • Bubble to clearly locate the placemarker on the map and to include the following from the database: Provider, Provider Address, status, URL

Course popup

Active in all cases not covered by Provider popup conditions.

  • Bubble to clearly locate the placemarker on the map and to include the following from the database: Provider, Provider address, Provider Status, Course name, Level and any information associated with the level. There are four different ‘Level’ fields. Where a value is NULL for any of those ‘Level’ fields, it should be invisible in the Pop-up bubble. In March's review of the data it looks as if there should be a maximum of two different ‘Levels’ in any record.
@gubuntu
Copy link
Contributor

gubuntu commented Jun 11, 2015

March:

The way the results are being generated, all courses which meet the search criteria AND which are based at the same Provider, are listed in the same bubble, rather than an individual bubble.

This is because the provider is the clickable entity in the Results list, not the individual course. This is reasonable given the volume of data but is not the only way to do it. The courses could be individually clickable creating individual bubbles.

There is merit in both approaches. If you are looking at Engineering at UCT, there are considerable advantages to seeing all the courses together which must be weighed against the presentational advantage of a smaller individual course bubble.

NB When the search criteria / multiple search term issue has been sorted out, this issue might fall away; more detailed searches lead to fewer results.

I recommend we leave this as it is for now.

...

  1. There is an ongoing discussion between TM and GF about content. Assuming we are not generating new functionality the exact detail is not important – but we must conclude the discussion.
  2. I personally think it could be helpful to have some static labels in the bubbles to describe specific content. E.g. ‘Address:’, ‘Course Name:’ etc.

There are some fields which are returning ‘NONE’. It may be that this is actual data (which needs to be cleaned in time), but if this is the system returning a response to a NULL field, the brief specifically asks for these not to appear

@lucernae
Copy link
Collaborator

I made the course now clickable instead of providers. This is because one provider can have multiple different campuses in different location, so the course now has the info of the campus location.

Abut the popup one, it doesn't seems clear to me. Does we will have provider popups too? This means somewhere we have to put campus information to be able to click and show the popup? As for the course popup we can implement it, we will have groups of popup available in the same location (because they has the same campus). Since they are in the same location, it might be strange to put the marker on the map, since many courses in the same campus will have the same location? What should we do with this? @cchristelis

@gubuntu gubuntu modified the milestone: Delivery of fiiinal release candidate Jul 14, 2015
@cchristelis
Copy link
Contributor Author

Hi @gubuntu,

I think this is no longer relevant in light of staging.feti.kartoza.com. Please re-open if we missed anything.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants