Skip to content
This repository has been archived by the owner on Aug 27, 2021. It is now read-only.

Search - Search by Attribute UI #45

Closed
NicoledeGreef opened this issue Jan 13, 2018 · 18 comments
Closed

Search - Search by Attribute UI #45

NicoledeGreef opened this issue Jan 13, 2018 · 18 comments
Assignees
Labels
enhancement New feature or request needs design
Milestone

Comments

@NicoledeGreef
Copy link
Collaborator

Very similar to existing DMF applications - see BC Parks as an example.

@cpspicer cpspicer added the enhancement New feature or request label Jan 18, 2018
@cpspicer cpspicer added this to the 1.1.0 milestone Jan 18, 2018
@cpspicer
Copy link

cpspicer commented Feb 5, 2018

Link to DMF in Test: https://test.apps.gov.bc.ca/ext/dmf-wizard/

@cpspicer
Copy link

cpspicer commented Feb 5, 2018

for board

@NicoledeGreef
Copy link
Collaborator Author

NicoledeGreef commented Feb 13, 2018

dependency on connectivity to MPCM endpoint. #80

@qqnluaq qqnluaq removed this from the 1.1.0 milestone Feb 19, 2018
@cpspicer cpspicer added this to the 1.3.0 milestone Mar 19, 2018
@cpspicer
Copy link

Lets set up a design session and discuss

@cpspicer cpspicer modified the milestones: 1.3.0, MVP Apr 24, 2018
@cpspicer
Copy link

Looking good. A few quick observations:

  1. Can't find option to have the app poll the service to determine live contents for drop down lists (maybe need instructions?)

  2. Would be nice to have an option for the author to choose an attribute to display along with the returned feature lists. Similar to DMF returned results list:
    image

  3. Tool is described as a 'Filter by' in the UI. Should we change this to 'Search using'?

image

  1. Execute text could be changed to 'Search'

  2. Could add a clear button in UI

@qqnluaq
Copy link
Collaborator

qqnluaq commented May 28, 2018

There is a configuration property to set the 'title attribute', ie the atttribute that is shown in the list above. This needs to be exposed in the admin UI.

@dhlevi
Copy link
Collaborator

dhlevi commented Jun 3, 2018

Re the first issue on @cpspicer comment above:

We've added a button in the admin UI to pull values. Currently it's activated for WMS features, and on a follow up next week we'll add in vector. I took a dive into the DMF code and it looks like it's configured to use the ArcGIS service calls to populate the list (geoprocess tools should let us get around the query limits), so we can use that same method (not always ideal as it is possible for it to miss values on large datasets, but I'll try and work out a sensible query).

However, because we want this to automatically update on the client side, I think this code should be removed from the Admin UI and moved into the client. On startup, the client could attempt to fetch a list that will always be up to date for a dropdown attribute.

@cpspicer
Copy link

cpspicer commented Jun 4, 2018

Agree code should be client side, with a mode option setting in the author.

@cpspicer
Copy link

QA Failure. App builder won't open the screen to configure Queries for a layer. e.g. Points of diversion. Clicking yields no response or error from the UI.

@dhlevi
Copy link
Collaborator

dhlevi commented Jun 16, 2018

@cpspicer Caught a bug in the admin ui related to the removal of surround editor panel that's affecting all modal dialogs. It's been patched, will do a build/deploy by Monday at the latest so you should see this issue fixed shortly.

@cpspicer
Copy link

cpspicer commented Jul 16, 2018

I can now create queries again - testing continues.

I can manually add drop down values for attributes, but the actual attribute values are blank when you click on the drop down to make a choice. The choices are there, and you can select a seemingly blank entry. If you hit return the drop down box opens, but if you click in a text field and hit return the query will run and return results from that 'invisible' drop down attribute.

Also, still no toggle in the UI to have a drop down box fed from the live dataset for MPCM datasets.

image

I do like the new title attribute. Thanks for adding!

@qqnluaq
Copy link
Collaborator

qqnluaq commented Jul 20, 2018

The title for a parameter select choice is now optional, and will be derived from value if missing.

https://github.com/bcgov/smk/wiki/SMK-Client-Configuration#choices-select-layer-query-parameter

@NicoledeGreef
Copy link
Collaborator Author

The queries are working well. The UI styling needs a little bit of tweaking. See "Assisted Living..." site". At 100% zoom level in chrome the Query and results panel is cut-off due to the number of attributes listed, and the Title is displaced upwards.

image

Would like to decrease the font-size for the Title and for the choices in the drop-down boxes as well. Can this be done through the .css?

@cpspicer
Copy link

cpspicer commented Sep 4, 2018

Linked to #193

@NicoledeGreef
Copy link
Collaborator Author

addition of scrollbar would perhaps alleviate some of the stretch trouble with the lengthy query form.

@cpspicer
Copy link

Good solution @qqnluaq

@cpspicer
Copy link

cpspicer commented Nov 2, 2018

link to #242

@cpspicer
Copy link

cpspicer commented Nov 2, 2018

closing this issue - moving 242 into the backlog

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request needs design
Projects
None yet
Development

No branches or pull requests

4 participants