search

VertNet / webapp

VertNet web application
8 stars 7 forks source link

Query Element Layout #606

Closed tucotuco closed 8 years ago

tucotuco commented 8 years ago

Already so much better than the production version. The Institution selection at the top is head-banging good. Why didn't we do that before?

Having it there makes it more obvious that another improvement would be to put all the record identifying query elements on the same row right under it. Thus, just under InstitutionCode, in order: CollectionCode, CatalogNumber, OccurrenceID, FieldNumber, RecordNumber

Then, on the next row, so that RecordNumber lies directly above RecordedBy: BasisOfRecord, Preparation, Sex, LifeStage, RecordedBy

robgur commented 8 years ago

+1

On Tue, Jul 12, 2016 at 6:42 AM, John Wieczorek notifications@github.com wrote:

Already so much better than the production version. The Institution selection at the top is head-banging good. Why didn't we do that before?

Having it there makes it more obvious that another improvement would be to put all the record identifying query elements on the same row right under it. Thus, just under InstitutionCode, in order: CollectionCode, CatalogNumber, OccurrenceID, FieldNumber, RecordNumber

Then, on the next row, so that RecordNumber lies directly above RecordedBy: BasisOfRecord, Preparation, Sex, LifeStage, RecordedBy

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/VertNet/webapp/issues/606, or mute the thread https://github.com/notifications/unsubscribe/AAcc7I_if8Ui4UoIVZ0hxHPhgcO5rB4cks5qU2-IgaJpZM4JKQ3p .

laurarussell commented 8 years ago

So am I removing License completely?

tucotuco commented 8 years ago

I didn't want to make that call, I just wanted to propose how the first three rows look. To me license is a medium-dubious one to want to query on.

On Tue, Jul 12, 2016 at 5:58 PM, Laura Russell notifications@github.com wrote:

So am I removing License completely?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/VertNet/webapp/issues/606#issuecomment-232093951, or mute the thread https://github.com/notifications/unsubscribe/AAcP67ETwlIRGpjB_eRreAxYH7b7Ks28ks5qU7mrgaJpZM4JKQ3p .

laurarussell commented 8 years ago

They only place for it now would be down on startdayofyear row. Has License doesn't really seem to me to be useful anymore because everybody will have a license so I didn't add it. It would seem like people would want to select/search for a specific license, but the content can be so varied. I'm fine with removing it. I added it previously just cause it was something we had talked about in the past.

tucotuco commented 8 years ago

It can be decided in the grand discussion on which of the many now-indexed fields to enable in queries.

On Tue, Jul 12, 2016 at 6:03 PM, Laura Russell notifications@github.com wrote:

They only place for it now would be down on startdayofyear row. Has License doesn't really seem to me to be useful anymore because everybody will have a license so I didn't add it. It would seem like people would want to select/search for a specific license, but the content can be so varied. I'm fine with removing it. I added it previously just cause it was something we had talked about in the past.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/VertNet/webapp/issues/606#issuecomment-232095191, or mute the thread https://github.com/notifications/unsubscribe/AAcP6z44TCrRQQuMDrrJdA9N34sesVOFks5qU7rMgaJpZM4JKQ3p .

laurarussell commented 8 years ago

Other fields have been moved, https://github.com/VertNet/webapp/commit/f3ff9f4355d3cb856da81ac6236615f16795ba05. Not really room for license now.

tucotuco commented 8 years ago

Like very much. Confirmed good.