CUNY Academic Commons

One remaining task here is that the "required" logic isn't 100% implemented.

Raymond Hoh wrote:

Received a report from Scott that the HTML5 error messaging is a little vague.

The default "Please select an item in the list" dialog can be improved:
https://commons.gc.cuny.edu/files/missing-input-error-message.png

I've changed the error message for the Campus field to Please select the campus(es) from the list. If this Commons space is not associated with a campus, select 'Not Applicable'

And the Primary Purpose error message to Please select a primary purpose for this Commons space or select 'Other' if not applicable.

(Code reference - https://github.com/cuny-academic-commons/cac/commit/c44c4b71dabb927e9f0f8c51848fe671984cc9f8)

I've attached screenshots. Let me know if the error message should be changed to something else.

Ray, we have defined all the tooltips in this document, please double check if the correct copy has been implemented:
https://docs.google.com/document/d/160nzBINvCEyeEVvnTeZETpLL7FphfJ5Yr28pptunCvM/edit?usp=sharing

Thanks for the link to the Google doc, Sonja.

I couldn't find any copy for error messages. What Scott brought up was the lack of specific error messaging when the Campus or Primary Purpose is blank and when the form is attempting to be submitted. The Google doc only appears to have copy for tooltips.

I'd be glad to change the copy for the error messages to something else.

Raymond Hoh wrote:

Thanks for the link to the Google doc, Sonja.

I couldn't find any copy for error messages. What Scott brought up was the lack of specific error messaging when the Campus or Primary Purpose is blank and when the form is attempting to be submitted. The Google doc only appears to have copy for tooltips.

I'd be glad to change the copy for the error messges to something else.

Thanks for clarifying. In that case the error messages you're added make sense from my perspective. We should add appropriate error messages for Academic term and Disciplinary cluster too then.

Please also make sure that dropdowns are consistent – some have a downward facing arrow, some don't. The arrow is a common indicator for drowdowns. So I'd opt to have arrows for all dropdowns. Thanks.

In that case the error messages you're added make sense from my perspective. We should add appropriate error messages for Academic term and Disciplinary cluster too then

That's a little harder to implement because those fields are conditional and depend on the "Primary Purpose" to be selected as "Teaching" first. If we want to implement this, this will have to be via javascript, which complicates things.

Please also make sure that dropdowns are consistent – some have a downward facing arrow, some don't. The arrow is a common indicator for drowdowns. So I'd opt to have arrows for all dropdowns. Thanks.

I believe the reason why some dropdowns do not have a dropdown arrow is because you cannot select multiple options for them. Those dropdowns that only require a single option will have the dropdown arrow. This is due to the Select2 JS library that we're using to render the dropdowns.

Raymond Hoh wrote:

I believe the reason why some dropdowns do not have a dropdown arrow is because you cannot select multiple options for them. Those dropdowns that only require a single option will have the dropdown arrow. This is due to the Select2 JS library that we're using to render the dropdowns.

That makes sense. Thanks for clarifying!