Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
126 commits
Select commit Hold shift + click to select a range
1d83c8d
Replace `spanx` by `col-md-x` and `row-fluid` by `row` for bootstrap …
bertpareyn Aug 29, 2013
9efacdf
Upgrade modals, buttons and navigation to Bootstrap 3
bertpareyn Aug 30, 2013
ea833eb
Upgrade index, dashboard and content profile widgets to Bootstrap 3, …
bertpareyn Sep 3, 2013
bf4886a
Upgrade search pages, uploadnewversion and register to bootstrap3
bertpareyn Sep 3, 2013
dd5f778
Upgrade the libraries to bootstrap 3.0
bertpareyn Sep 3, 2013
3258db3
Update form validation, Update progress indication
bertpareyn Sep 4, 2013
22ee8dd
Update notify to bootstrap 3.0, Updates to the content profile page
bertpareyn Sep 4, 2013
5f665e8
Make large list options flexible
bertpareyn Sep 5, 2013
e56b767
Address large list options visual issues
bertpareyn Sep 5, 2013
2578e3e
Upgrade the admin UI to bootstrap 3.0
bertpareyn Sep 5, 2013
36f859a
Upgrade Admin UI modals to Bootstrap 3.0
bertpareyn Sep 5, 2013
827935c
Adjust footer and header in Admin UI
bertpareyn Sep 5, 2013
5176b4f
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Sep 9, 2013
0486c9f
Refinements to the responsive left hand navigation
bertpareyn Sep 9, 2013
080e821
Remove unnecessary page CSS
bertpareyn Sep 9, 2013
4556728
Merge origin/master and fix conflicts
bertpareyn Sep 11, 2013
6b2edad
Merge origin/master and fix conflicts
bertpareyn Sep 25, 2013
aa6d504
Address Bootstrap 3 top navigation issues
bertpareyn Sep 26, 2013
69ad820
More top navigation and register widget work
bertpareyn Sep 26, 2013
9f54c5a
Activity, notifications and lists adjustments for Bootstrap 3
bertpareyn Sep 27, 2013
d798728
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Sep 30, 2013
299e5fb
Further improvements to Bootstrap 3 upgrade
bertpareyn Sep 30, 2013
df7d478
Bootstrap tweaks
bertpareyn Oct 1, 2013
79cc5ec
Admin UI fixes for bootstrap 3 upgrade
bertpareyn Oct 4, 2013
4f43393
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Oct 7, 2013
78d47dc
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Oct 10, 2013
8707aaf
Align left hand navigation with top of content
bertpareyn Oct 10, 2013
6be0e04
Merge origin/master and fix conflicts
bertpareyn Oct 17, 2013
7696f44
Merge origin/master and fix conflicts
bertpareyn Oct 21, 2013
1e0403c
Merge origin/master and fix conflicts, update various widgets to comp…
bertpareyn Oct 30, 2013
772a712
Fix qunit error
bertpareyn Oct 30, 2013
2606fcf
Upgrade Bootstrap to 3.0.1
bertpareyn Oct 31, 2013
9c19b6d
Merge origin/master and fix conflicts, bring new components up to dat…
bertpareyn Oct 31, 2013
f9fd288
Merge origin/master and fix conflicts
bertpareyn Oct 31, 2013
c87d6d9
Merge origin/master and fix conflicts
bertpareyn Nov 6, 2013
f3fa388
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Nov 14, 2013
616e8c5
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Nov 18, 2013
6826b24
Merge origin/master and fix conflicts
bertpareyn Nov 28, 2013
737bc2c
Initial implementation of responsive left hand navigation and clips t…
bertpareyn Dec 3, 2013
9767f75
Hide clips on group page when added to the left hand navigation
bertpareyn Dec 3, 2013
a6b3fb0
Merge origin/master and fix conflicts
bertpareyn Dec 3, 2013
2398523
Upgrade Bootstrap from 3.0.1 --> 3.0.2
bertpareyn Dec 3, 2013
54bf33b
Finish implementing responsive content left hand nav basics, componen…
bertpareyn Dec 4, 2013
d0e1421
Finish implementation of responsive search page
bertpareyn Dec 5, 2013
940a28d
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Dec 5, 2013
2daa779
Add in responsive nav animation, improvements to the search page sear…
bertpareyn Dec 5, 2013
828e003
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Dec 6, 2013
635cb8c
Refinements to lhnav margins/paddings based on feedback, responsive r…
bertpareyn Dec 6, 2013
e66eede
Upgrade to Bootstrap 3.0.3
bertpareyn Dec 6, 2013
e13913f
Implement responsive clips
bertpareyn Dec 6, 2013
d5113cb
Responsive work in: admin area, product video, top navigation and sig…
bertpareyn Dec 9, 2013
f4436c4
Upgrade bootstrap.js to 3.0.3, update unfollow modal, threedot all th…
bertpareyn Dec 9, 2013
417cd7a
Merge origin/master and fix conflicts
bertpareyn Dec 9, 2013
405e0c1
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Dec 10, 2013
bcd601e
Responsive list options, responsive notifications, fix minor issue in…
bertpareyn Dec 11, 2013
5790771
Responsive notifications, Responsive search on index
bertpareyn Dec 11, 2013
35fb56f
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Dec 13, 2013
a93df1e
Merge origin/master and fix conflicts
bertpareyn Dec 16, 2013
e1a7308
Merge origin/master and fix conflicts
bertpareyn Dec 18, 2013
924e34b
Fix bootstrap3 upgrade visual regressions in register, top navigation…
bertpareyn Dec 20, 2013
6279587
Bootstrap 3 follow-up
bertpareyn Jan 6, 2014
a4da57b
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 6, 2014
61449cb
Content profile followup, comments, manageaccess, documentpreview, et…
bertpareyn Jan 6, 2014
ba431d4
Follow up on bootstrap error pages, admin UI, docs UI and search
bertpareyn Jan 6, 2014
bc71004
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 6, 2014
0083548
Follow up on bootstrap 3 discussion page
bertpareyn Jan 7, 2014
00d9fb1
Enable share in the left hand nav, tweaks for mobile
bertpareyn Jan 7, 2014
816f2dd
Don't show productvideo modal on mobile, instead embed video straight…
bertpareyn Jan 7, 2014
442d4c6
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 7, 2014
9988371
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 7, 2014
c280f55
Bootstrap 3 modifications for the changepic widget
bertpareyn Jan 8, 2014
38871bb
Left hand navigation tweaks
bertpareyn Jan 8, 2014
6c13303
Replace div with footer and main element
bertpareyn Jan 8, 2014
86cf090
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 8, 2014
cd4e8a1
Merge origin/master and fix conflicts
bertpareyn Jan 8, 2014
d3205a3
Merge origin/master and fix conflicts
bertpareyn Jan 9, 2014
6de2c03
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 13, 2014
62a7241
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 14, 2014
375c78a
Use CSS flexbox to improve threedots
bertpareyn Jan 14, 2014
56e21e4
Fix broken font awesome reference in build (update paths after upgrad…
bertpareyn Jan 14, 2014
cae55a6
Follow up on bootstrap 3 regressions
bertpareyn Jan 15, 2014
66cfeb8
break-word on media-body to avoid overflow in comments
bertpareyn Jan 15, 2014
7cde329
Merge origin/master and fix conflicts
bertpareyn Jan 15, 2014
d889b70
Break word in activities as well
bertpareyn Jan 16, 2014
dc9b09c
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 16, 2014
aea71c5
Follow up on Bootstrap3 review
bertpareyn Jan 17, 2014
5e7d8d0
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 17, 2014
65cea16
(Issue 3384) Use id for link container rather than positioning
mrvisser Jan 17, 2014
b3fe67f
Merge pull request #15 from mrvisser/issue-3384
bertpareyn Jan 17, 2014
511a023
Bootstrap 3 followup
bertpareyn Jan 17, 2014
dc6e0d6
Bootstrap 3 followup
bertpareyn Jan 18, 2014
94db395
Bootstrap 3 followup
bertpareyn Jan 18, 2014
3bd8b46
Follow up on the left hand navigation review
bertpareyn Jan 18, 2014
90dd75a
Remove lingering event
bertpareyn Jan 18, 2014
483b7d5
Follow up on bootstrap 3 CSS review
bertpareyn Jan 19, 2014
c1146d7
Follow up on bootstrap.notify, docs and minor issues
bertpareyn Jan 19, 2014
e4de13b
Bootstrap 3 Left hand navigation refinements
bertpareyn Jan 19, 2014
7bf892f
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 19, 2014
06456d4
Merge origin/master and fix conflicts
bertpareyn Jan 21, 2014
7bec567
Bootstrap followup
bertpareyn Jan 21, 2014
7eb845f
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 24, 2014
1bf8cf3
Bootstrap 3 followup
bertpareyn Jan 24, 2014
d662d33
Bootstrap 3 followup
bertpareyn Jan 24, 2014
3258f85
Add documentation for the oae-lhnavigation component
bertpareyn Jan 27, 2014
d8f1c19
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 27, 2014
832a7e5
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 28, 2014
6033c5e
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Jan 31, 2014
e31f301
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Feb 3, 2014
5c442dc
Follow up on Bootstrap 3 review
bertpareyn Feb 3, 2014
2c93bfd
Merge origin/master and fix conflicts
bertpareyn Feb 3, 2014
7f5650a
Follow up on Bootstrap 3 review
bertpareyn Feb 3, 2014
f12a52f
Follow up on Bootstrap 3 review
bertpareyn Feb 3, 2014
2ba701a
Follow up on Bootstrap 3 review
bertpareyn Feb 3, 2014
927e1be
Fix my library display issues on IE9
Feb 5, 2014
d84a37f
Merge pull request #59 from timdegroote/ie9-fix
bertpareyn Feb 5, 2014
80bd7ca
Pass in baseURL in content and discussion left hand navigation
bertpareyn Feb 5, 2014
513f17f
Follow up on Bootstrap 3 review
bertpareyn Feb 5, 2014
969920a
Use CSS to hide the lhnav collapse button
bertpareyn Feb 5, 2014
bcb1260
Optimize re-rendering content and discussion previews when updates ha…
bertpareyn Feb 5, 2014
ff995f8
Improve selector for clearing preview container
bertpareyn Feb 5, 2014
0455f37
Update jquery spectrum plugin
bertpareyn Feb 10, 2014
75ab3b8
Merge origin/master and fix conflicts
bertpareyn Feb 10, 2014
b67194f
Show the lhnav toggle button for anonymous users on user and group pages
bertpareyn Feb 10, 2014
6388af8
Merge origin/master and fix conflicts
bertpareyn Feb 12, 2014
f889d31
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Feb 12, 2014
2a81fcd
Merge branch 'master' of https://github.com/sakaiproject/3akai-ux int…
bertpareyn Feb 14, 2014
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
72 changes: 44 additions & 28 deletions admin/css/admin.css
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,10 @@
*/

body {
margin: 0;
padding: 0;
}

#admin-header-container, #admin-content-container {
#admin-header-container {
padding: 0 30px;
}

Expand All @@ -30,7 +30,7 @@ body {
}

#admin-header-container #admin-header-content {
background: url('/shared/oae/img/oae-logo.png') no-repeat left center transparent;
background: url('/shared/oae/img/oae-logo.png') no-repeat 15px center transparent;
Copy link
Contributor

Choose a reason for hiding this comment

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

Please ignore previous comment here

background-size: 60px;
min-height: 70px;
}
Expand All @@ -47,14 +47,18 @@ body {
font-size: 22px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Visual regressions on the admin UI:

  • The footer is not displaying correctly
  • The login form labels are bold and are too close to the input field
  • The spacing between the fields and the submit button is inconsistent, we should probably move the button a bit further down
  • In small mode, the left hand navigation being on top of the content isn't working very well. Can we do a collapsable lhnav instead, similar to the other ones?
  • There is too much spacing between the lhnavigation items
  • The overall page titles are a different font

Copy link
Contributor

Choose a reason for hiding this comment

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

Visual regressions in the tenant overview:

  • The create new tenant form has the same display issue as the login form
  • There is too much spacing underneath the text in the confirm dialog
  • It might make sense to make cancel a link and float it to the left, that seems to shrink more gracefully

Copy link
Contributor

Choose a reason for hiding this comment

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

Visual regressions in the configuration pane:

  • The save buttons are too close to the fields
  • The labels are bold and too close to the fields

Copy link
Contributor

Choose a reason for hiding this comment

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

Visual regressions in the skinning pane:

  • There is spacing between the items inside of the same skinning block
  • The reset icon doesn't display nicely on smaller screens
  • The Save skin button is too far to the right
  • The color picker is not displaying correctly
  • The labels are bold
  • The Skin values title needs a bit more margin-bottom

font-weight: normal;
margin: 0;
padding: 15px 0 0 80px;
padding: 18px 0 0 80px;
}


/*************
** CONTENT **
*************/

#admin-content {
margin-bottom: 100px;
}

#admin-content-container {
padding-top: 20px;
padding-bottom: 120px;
Expand All @@ -68,6 +72,10 @@ body {
margin-bottom: 20px;
}

.admin-container-striped form {
margin: 20px 0;
}

/* Tenant list */

.table-striped td.admin-tenants-visit {
Expand Down Expand Up @@ -114,29 +122,32 @@ td.jeditable-container:hover {
background: linear-gradient(to bottom, #DDD 0%,#CCC 100%) !important; /* W3C */
}

td.jeditable-container form input {
height: 25px !important;
}

td form {
margin: 0;
}

/* Module configuration */

.admin-module-configuration-container input[type="checkbox"],
.admin-module-configuration-container .admin-module-configuration-toggle-button i {
float:left;
margin-right:12px;
.admin-module-configuration-container {
margin: 0;
}

.admin-module-configuration-container .form-horizontal .control-label {
width: 225px;
.admin-module-configuration-container form {
margin: 0 0 20px;
}

.admin-module-configuration-container .form-horizontal .controls {
margin-left: 250px;
.admin-module-configuration-container .admin-module-configuration-toggle-button i {
float:left;
margin-right:12px;
}

.admin-module-configuration-container h4 {
margin-top: 20px;
margin-bottom: 20px;
margin-bottom: 7px;
}

.admin-module-configuration-container h4 span,
Expand All @@ -145,16 +156,15 @@ td form {
font-weight: normal;
}


.admin-container-striped form {
margin: 20px 0;
}
.admin-container-striped > div:nth-child(2n+1),
.admin-container-striped > div:nth-child(2n+2) {
border-top: 1px solid #DDD;
max-width: none;
padding: 12px;
}
.admin-container-striped form .form-group {
margin-left: 0;
}

/* Skinning configuration */

Expand All @@ -168,20 +178,26 @@ td form {
margin-top: 20px;
}

#admin-skinning-container .control-group {
margin-bottom: 0px;
#admin-skinning-container #admin-skinning-actions {
margin-top: 30px;
}

#admin-skinning-container .control-label {
width: 350px;
#admin-skinning-form .form-group {
margin: 0;
}

#admin-skinning-container .controls {
margin-left: 370px;
/* Override bootstrap box sizing for the spectrum plugin https://github.com/bgrins/spectrum/issues/175 */

.sp-container *,
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes the spectrum plugin was displaying in a weird way.

Copy link
Contributor

Choose a reason for hiding this comment

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

Might be worth creating a ticket in the spectrum repository saying that it doesn't work well with boostrap

Copy link
Contributor

Choose a reason for hiding this comment

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

Is this something that we did? If so, we should add the link to the ticket in here

.sp-container *:after,
.sp-container *:before {
-webkit-box-sizing: content-box;
-moz-box-sizing: content-box;
box-sizing: content-box;
}

#admin-skinning-container #admin-skinning-actions {
margin-top: 30px;
.sp-input-container .sp-input {
background-color: #FFF;
}

/************
Expand All @@ -198,13 +214,13 @@ td form {
}

#admin-footer-content {
background: url("/shared/oae/img/oae-logo-white.png") no-repeat scroll left 20px transparent;
background: url("/shared/oae/img/oae-logo-white.png") no-repeat scroll 15px center transparent;
background-size: 70px;
min-height: 75px;
min-height: 80px;
}

#admin-footer-container ul {
margin: 30px 0 10px 50px;
margin: 30px 0 0 50px;
}

#admin-footer-container ul li {
Expand Down
259 changes: 131 additions & 128 deletions admin/index.html

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion admin/js/admin.skin.js
Original file line number Diff line number Diff line change
Expand Up @@ -261,7 +261,7 @@ define(['exports', 'jquery', 'underscore', 'oae.core', 'jquery.spectrum'], funct
* value back to its previous value.
*/
var revertSkinValue = function() {
var $input = $('input', $(this).parent());
var $input = $('input', $(this).parents('.form-group'));
var defaultValue = defaultSkin[$input.attr('name')];
// If the variable is a color, we use the set method provided by jQuery spectrum
if ($input.attr('data-type') === 'color') {
Expand Down
8 changes: 4 additions & 4 deletions admin/js/admin.util.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,10 @@ define(['exports', 'jquery', 'oae.core'], function(exports, $, oae) {
var toggleInternationalizableFieldContainer = exports.toggleInternationalizableFieldContainer = function() {
// Define the target container
var $targetContainer = $(this).parents('.admin-internationalizable-text-container');
// Hide all control groups except the first one (language select) in the target container
$targetContainer.find('.control-group').hide();
$targetContainer.find('.control-group:first-child').show();
// Show the selected language control group in the target container
// Hide all form groups except the first one (language select) in the target container
$targetContainer.find('.form-group').hide();
$targetContainer.find('.form-group:first-child').show();
// Show the selected language form group in the target container
$targetContainer.find('div[data-id="' + $(this).val() + '"]').show();
};

Expand Down
7 changes: 7 additions & 0 deletions docs/css/doc.css
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@

html, body {
margin: 0;
padding: 0;
}
#doc-header-container, #doc-content-container {
padding: 0 30px;
Expand Down Expand Up @@ -105,9 +106,15 @@ td form {
display: block;
margin: 0 0 -1px;
padding: 8px 14px;
word-break: break-word;
}
.bs-docs-sidenav > li > a > span {
display: block;
margin-right: 22px;
}
.bs-docs-sidenav .icon-chevron-right {
margin-right: -6px;
margin-top: -5px;
opacity: 0.25;
}
.nav > li > a:hover {
Expand Down
15 changes: 8 additions & 7 deletions docs/index.html
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
<!DOCTYPE html>
<html lang="en">
<head>
<meta name="viewport" content="width=device-width, user-scalable=no">
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1" />
<noscript>
Expand All @@ -10,7 +11,7 @@
<!-- CORE CSS -->
<link rel="stylesheet" type="text/css" href="/shared/oae/css/oae.core.css" />
<link rel="stylesheet" type="text/css" href="/api/ui/skin" />

<!-- PAGE CSS -->
<link href="/docs/css/doc.css" type="text/css" rel="stylesheet"/>
</head>
Expand All @@ -22,20 +23,20 @@ <h1>API Reference</h1>
</div>
</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

Visual regressions on the docs page:

  • The titles in the left hand navigation are a different font
  • The arrows in the left hand navigation are not vertically centred
  • Titles inside of the docs content are a different font

<!-- CONTENT -->
<div id="doc-content-container">
<div class="row-fluid">
<div id="doc-modules-container" class="span3"><!----></div>
<div id="doc-module-container" class="span9"><!----></div>
<main id="doc-content-container" role="main">
<div class="row">
<div id="doc-modules-container" class="col-sm-3"><!----></div>
<div id="doc-module-container" class="col-sm-9"><!----></div>
</div>
</div>
</main>

<div id="doc-modules-template"><!--
{macro renderModules(title, type, modules)}
<h5>${title}</h5>
<ul class="nav nav-list bs-docs-sidenav">
{for module in modules}
<li data-id="${module}" data-type="${type}">
<a href="/docs/${type}/${module}"><i class="icon-chevron-right pull-right"></i> ${module}</a>
<a href="/docs/${type}/${module}"><i class="icon-chevron-right pull-right"></i> <span>${module}</span></a>
</li>
{/for}
</ul>
Expand Down
124 changes: 64 additions & 60 deletions node_modules/oae-core/accountpreferences/accountpreferences.html

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading