Merge lp://staging/~whosdaz/ubuntu-html5-theme/googlecodein into lp://staging/ubuntu-html5-theme

Proposed by Matthew Allen
Status: Merged
Approved by: David Barth
Approved revision: 218
Merged at revision: 212
Proposed branch: lp://staging/~whosdaz/ubuntu-html5-theme/googlecodein
Merge into: lp://staging/ubuntu-html5-theme
Diff against target: 551 lines (+94/-105)
4 files modified
0.1/ambiance/css/appTemplate.css (+48/-65)
0.1/ambiance/css/sass/default.scss (+3/-1)
0.1/ambiance/css/sass/toggles.scss (+37/-34)
examples/html5-theme/widgets/Toggles.html (+6/-5)
To merge this branch: bzr merge lp://staging/~whosdaz/ubuntu-html5-theme/googlecodein
Reviewer Review Type Date Requested Status
David Barth Approve
Review via email: mp+280386@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
David Barth (dbarth) wrote :
Download full text (18.1 KiB)

I approved the task in GoogleCodeIn. Well done, and thanks for your
contribution Matthew !

On Sat, Dec 12, 2015 at 1:02 PM, MatthewAllen <email address hidden> wrote:

> MatthewAllen has proposed merging
> lp:~whosdaz/ubuntu-html5-theme/googlecodein into lp:ubuntu-html5-theme.
>
> Requested reviews:
> Ubuntu HTML5 Theme Developers (ubuntu-html5-theme-devs)
> Related bugs:
> Bug #1522819 in Ubuntu HTML5 UI SDK: "HTML5 Gallery - Toggles section"
> https://bugs.launchpad.net/ubuntu-html5-theme/+bug/1522819
>
> For more details, see:
>
> https://code.launchpad.net/~whosdaz/ubuntu-html5-theme/googlecodein/+merge/280386
> --
> Your team Ubuntu HTML5 Theme Developers is requested to review the
> proposed merge of lp:~whosdaz/ubuntu-html5-theme/googlecodein into
> lp:ubuntu-html5-theme.
>
> === modified file '0.1/ambiance/css/appTemplate.css'
> --- 0.1/ambiance/css/appTemplate.css 2015-01-29 15:14:59 +0000
> +++ 0.1/ambiance/css/appTemplate.css 2015-12-12 12:01:25 +0000
> @@ -672,7 +672,7 @@
> }
>
> .textfield.clear:focus {
> - background: url(artwork/icon_clear@20.png) no-repeat 97% center white;
> + background: url(artwork/icon_clear@20.png) no-repeat 97% center #fff;
> background-size: 12%;
> }
>
> @@ -785,7 +785,6 @@
> from {
> -webkit-transform: rotate(1deg);
> }
> -
> to {
> -webkit-transform: rotate(360deg);
> }
> @@ -794,11 +793,9 @@
> 0% {
> opacity: 0;
> }
> -
> 50% {
> opacity: 1;
> }
> -
> 100% {
> opacity: 0;
> }
> @@ -928,43 +925,33 @@
> 0% {
> -webkit-transform: translate3d(0, 0, 0);
> }
> -
> 10% {
> -webkit-transform: translate3d(-20px, 0, 0);
> }
> -
> 20% {
> -webkit-transform: translate3d(20px, 0, 0);
> }
> -
> 30% {
> -webkit-transform: translate3d(-20px, 0, 0);
> }
> -
> 40% {
> -webkit-transform: translate3d(20px, 0, 0);
> }
> -
> 50% {
> -webkit-transform: translate3d(-20px, 0, 0);
> }
> -
> 60% {
> -webkit-transform: translate3d(20px, 0, 0);
> }
> -
> 70% {
> -webkit-transform: translate3d(-20px, 0, 0);
> }
> -
> 80% {
> -webkit-transform: translate3d(20px, 0, 0);
> }
> -
> 90% {
> -webkit-transform: translate3d(-20px, 0, 0);
> }
> -
> 100% {
> -webkit-transform: translate3d(0, 0, 0);
> }
> @@ -973,43 +960,33 @@
> 0% {
> -webkit-transform: translate3d(0, 0, 0);
> }
> -
> 10% {
> -webkit-transform: translate3d(-20px, 0, 0);
> }
> -
> 20% {
> -webkit-transform: translate3d(20px, 0, 0);
> }
> -
> 30% {
> -webkit-transform: translate3d(-20px, 0, 0);
> }
> -
> 40% {
> -webkit-transform: translate3d(20px, 0, 0);
> }
> -
> 50% {
> -webkit-transform: translate3d(-20px, 0, 0);
> }
> -
> 60% {
> -webkit-transform: translate3d(20px, 0, 0);
> }
> -
> 70% {
> -webkit-transform: translate3d(-20px, 0, 0);
> }
> -
> 80% {
> -webkit-transform: translate3d(20px, 0, 0);
> }
> -
> 90% {
> -webkit-transform: translate3d(-20px, 0,...

Revision history for this message
David Barth (dbarth) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches