Skip to content
This repository has been archived by the owner on Jun 18, 2024. It is now read-only.

Panel upgrades #152

Merged
merged 5 commits into from
Sep 27, 2016
Merged

Panel upgrades #152

merged 5 commits into from
Sep 27, 2016

Conversation

msecret
Copy link

@msecret msecret commented Sep 21, 2016

Changes typography in the panel to match new designs, adding colors,
etc.

Should be minor version change.

Changes typography in the panel to match new designs, adding colors,
etc.
Marco Segreto and others added 3 commits September 20, 2016 17:31
Moves confirmation box from actions to it's own component and
fixes the combinations of colors for button styles.

TODO actions need to be refactored.
@juliaelman juliaelman self-assigned this Sep 22, 2016
@@ -2,6 +2,7 @@
@import 'actions.scss';
@import 'activity_log.scss';
@import 'billboard.scss';
@import 'confirm';
Copy link
Contributor

Choose a reason for hiding this comment

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

confirm.scss

width: 100%;
}

.confirm-nexto {
Copy link
Contributor

Choose a reason for hiding this comment

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

confirm-next-to?

Copy link
Author

Choose a reason for hiding this comment

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

It should be that because that would mean to is an element or modifier, wheresas "next to" is the modifier.

}

/* TODO refactor all actions */
Copy link
Contributor

@jcscottiii jcscottiii Sep 26, 2016

Choose a reason for hiding this comment

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

refactor how so? or put a link to an issue here.
thinking about if another dev came in, would they know what to do

@jcscottiii jcscottiii merged commit 574690c into master Sep 27, 2016
@jcscottiii jcscottiii deleted the ms-panel_typography branch September 27, 2016 13:13
@juliaelman juliaelman removed their assignment Sep 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants