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

Closes #333 - Negative disability feedback #357

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

reekypete
Copy link
Collaborator

No description provided.

Copy link
Contributor

@techieshark techieshark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments in review. I noticed a .least-text issue.


<p class="disclaimer lesser-text"><a id="specialist-support-click" href="javascript:return false">Information for specialist support</a></p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


<p class="disclaimer lesser-text"><a id="specialist-support-click" href="javascript:return false">Information for specialist support</a></p>
<div id="specialist-support-info" style="display:none">
<p class="disclaimer lesser-text least-text">The department provides a range of specialist support classes in regular and special schools. Access to these classes is through a separate placement panel process. Parents and carers wanting to enrol their child in a specialist support class should use the School Finder to locate their local public school and contact the principal directly. The principal will provide advice about the process to seek enrolment in a specialist support class.</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the 'least-text' class? .lesser-text is defined here: https://github.com/CodeforAustralia/school-finder/blob/2388e6930b8fa43fc4b0f73356a589ed357ca0a9/css/style.css but I don't see least-text

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do like the content we're providing parents and principals here - it's specific and clear.

<p class="disclaimer lesser-text"><a id="specialist-support-click" href="javascript:return false">Information for specialist support</a></p>
<div id="specialist-support-info" style="display:none">
<p class="disclaimer lesser-text least-text">The department provides a range of specialist support classes in regular and special schools. Access to these classes is through a separate placement panel process. Parents and carers wanting to enrol their child in a specialist support class should use the School Finder to locate their local public school and contact the principal directly. The principal will provide advice about the process to seek enrolment in a specialist support class.</p>
<p class="disclaimer lesser-text least-text">Information about this process is available at <a href="https://education.nsw.gov.au/disability-learning-and-support/programs-and-services" target="_blank">Programs and Services</a>. For the current list of regular and special schools with specialist support classes, visit the <a href="https://data.cese.nsw.gov.au/data/dataset/specialist-support-classes-by-school-and-support-needs-type" target="_blank">Datahub.</a></p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

again, 'least-text'?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great plug for the Datahub!

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

Successfully merging this pull request may close these issues.

2 participants