Skip to content
This repository has been archived by the owner on Dec 22, 2023. It is now read-only.

Materialize my-profile #72

Merged
merged 11 commits into from
Nov 9, 2016
Merged

Materialize my-profile #72

merged 11 commits into from
Nov 9, 2016

Conversation

thevoiceofzeke
Copy link
Contributor

@thevoiceofzeke thevoiceofzeke commented Oct 26, 2016

Opened this PR early -- using it to keep track of progress and note anything left to do so I don't forget things day to day.

In this PR:

  • Converted emergency info view to material
  • Removed unused CSS
  • Fixed a typo in emergencyInfo route name
  • Updated to use uw-frame 2.9.0
  • Updated the readme with instructions for running locally
  • Converted admin lookup layout to material

Note: my-profile-webapp/pom.xml was included due to an accidental commit. All changes are whitespace only and can be ignored

To do:

  • Add name attributes and ng-messages error help text to all form inputs (backlogged: MUMPFL-145)

Screenshots

screen shot 2016-11-09 at 11 08 40 am
screen shot 2016-11-09 at 11 09 00 am
screen shot 2016-11-09 at 11 09 15 am
screen shot 2016-11-09 at 11 09 24 am

@thevoiceofzeke thevoiceofzeke changed the title [WIP] Materialize my-profile Materialize my-profile Nov 7, 2016
@Doug-Reed
Copy link
Contributor

👍

Copy link
Member

@apetro apetro left a comment

Choose a reason for hiding this comment

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

Consider supporting pull request with screenshots.

<p ng-if="contact.relationship">Relationship: {{ contact.relationship }}</p>
<p ng-if="contact.comments">Comments: {{ contact.comments }}</p>
<div layout="row" ng-hide="contact.readOnly" class="buttons-small">
<md-button class="md-accent md-raised" ng-click="addEdit()">Add another</md-button>
Copy link
Member

Choose a reason for hiding this comment

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

In another spot, "Add another" is nuanced for screen readers with a clarifying aria-label (add another what?) Seemed like a good move?

<compress>true</compress>
<includes>
<include>my-app.less</include>
<include>admin.less</include>
Copy link
Member

Choose a reason for hiding this comment

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

Noting that this is the only substantive change to pom.xml; everything else changing in pom.xml is just whitespace.

@thevoiceofzeke thevoiceofzeke merged commit b88ada9 into UW-Madison-DoIT:master Nov 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants