-
Notifications
You must be signed in to change notification settings - Fork 5
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
UTCT-24: UTC Branding on front and back ends #44
UTCT-24: UTC Branding on front and back ends #44
Conversation
Signed-off-by: Chris Gilligan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good:
- Power C
- UTCentric frontend words
Signed-off-by: Chris Gilligan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
I love it, but just curious, are we allowed to cut the power c in half in the background? |
@Bridget, IDK but it can be moved up or down if required. It's just using the default positioning of the default logo. |
So, I did verify... the Power C is not allowed to be on a background that bisects it for brand compliance. Also, it was requested that the whole horizontal logo be presented above the white field, not just the power C. Otherwise, I think it looks FANTASTIC! |
|
Signed-off-by: Chris Gilligan <[email protected]>
Signed-off-by: Chris Gilligan <[email protected]>
# enter Github branch ID, UTC-1234: Sentence about commit. Signed-off-by: Chris Gilligan <[email protected]>
Signed-off-by: Chris Gilligan <[email protected]>
Signed-off-by: Chris Gilligan <[email protected]>
Signed-off-by: Chris Gilligan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
- Frontend styles and assets to stylize for UTC
- Backend styles and assets to stylize for UTC
@@ -46,10 +47,10 @@ $border-radius: 6px; | |||
//@import "../../node_modules/bootstrap/scss/breadcrumb"; | |||
//@import "../../node_modules/bootstrap/scss/pagination"; | |||
//@import "../../node_modules/bootstrap/scss/badge"; | |||
//@import "../../node_modules/bootstrap/scss/alert"; | |||
@import "../../node_modules/bootstrap/scss/alert"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for frontend alerts such as "shortcut reserved or already in use"
//@import "../../node_modules/bootstrap/scss/progress"; | ||
//@import "../../node_modules/bootstrap/scss/list-group"; | ||
//@import "../../node_modules/bootstrap/scss/close"; | ||
@import "../../node_modules/bootstrap/scss/close"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for dismissing frontend alerts.
@@ -1,5 +1,6 @@ | |||
@charset "UTF-8"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Compiled frontend styles; built from frontend tooling via npm-sass.
@@ -1,5 +1,5 @@ | |||
<script src="https://cdn.jsdelivr.net/npm/[email protected]/dist/js/bootstrap.bundle.min.js" integrity="sha384-MrcW6ZMFYlzcLA8Nl+NtUVF0sA7MsXsP1UyJoMp4YLEuNSfAP+JcXn/tWtIaxVXM" crossorigin="anonymous"></script> | |||
|
|||
<script src="https://cdn.jsdelivr.net/npm/@popperjs/[email protected]/dist/umd/popper.min.js" integrity="sha384-oBqDVmMz9ATKxIep9tiCxS/Z9fNfEXiDAYTujMAeBAsjFuCZSmKbSSUnQlmh/jp3" crossorigin="anonymous"></script> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
popper.js is no longer part of main bootstrap.min.js. Needed for some frontend transitions such as the Customize Link accordion.
@@ -86,21 +86,21 @@ function shorten() { | |||
</div> | |||
|
|||
<div class="card-body px-5 pb-5"> | |||
<h2 class="text-uppercase text-center">Your shortened link</h2> | |||
<h2 class="text-uppercase text-center">Your short link</h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is shorter.
@@ -0,0 +1,8 @@ | |||
User-agent: * |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discourage snooping bots.
Signed-off-by: Chris Gilligan <[email protected]>
Signed-off-by: Chris Gilligan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, Chris and team!
Signed-off-by: Chris Gilligan <[email protected]>
Signed-off-by: Chris Gilligan <[email protected]>
Signed-off-by: Chris Gilligan <[email protected]>
Signed-off-by: Chris Gilligan <[email protected]>
Signed-off-by: Chris Gilligan <[email protected]>
Signed-off-by: Chris Gilligan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
This has been applied to https://gotest.utc.edu, so please have a look.
Screenshots: