search

camelot-project / frontend

The frontend: The web-facing server software for CAMELOT
BSD 3-Clause "New" or "Revised" License
2 stars 14 forks source link

User-friendlied the layout of parse_file.html. NOTE: "required" input fields only work for "username". #92

Closed jmdkastro closed 9 years ago

keflavich commented 9 years ago

Could you provide a screenshot of this?

For the required bug, I think it does work for others, but the Submit button will turn green after the username field is filled

jmdkastro commented 9 years ago

parse_file

keflavich commented 9 years ago

The "Units" being required is somewhat ambiguous, but I think you should leave it as you've done here. #91 should include the solution to some units being required.

jmdkastro commented 9 years ago

Why would a DOI/URL always be required? An ADS ID should be sufficient, and from that an ADS URL can be generated. Providing an ADS ID and and ADS URL would be redundant I'd think.

keflavich commented 9 years ago

It's better to have both if possible, since that provides for a potential validation.

jmdkastro commented 9 years ago

As the stupid end-user which I am, I find that highly confusing. In that case, we should simply ask for the ADS URL.