Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29339552: Issue 3895 - Fix size of dialog close buttons on the options page (Closed)

Created:
April 7, 2016, 10:35 a.m. by Sebastian Noack
Modified:
April 15, 2016, 4:54 p.m.
Reviewers:
Thomas Greiner
Visibility:
Public.

Description

Issue 3895 - Fix size of dialog close buttons on the options page

Patch Set 1 #

Total comments: 2

Patch Set 2 : Addressed comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M skin/options.css View 1 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 4
Sebastian Noack
April 7, 2016, 10:36 a.m. (2016-04-07 10:36:39 UTC) #1
Thomas Greiner
https://codereview.adblockplus.org/29339552/diff/29339553/skin/options.css File skin/options.css (right): https://codereview.adblockplus.org/29339552/diff/29339553/skin/options.css#newcode1203 skin/options.css:1203: background-position: -9px -32px; Detail: These coordinates should be `-10px ...
April 7, 2016, 5:40 p.m. (2016-04-07 17:40:11 UTC) #2
Sebastian Noack
On 2016/04/07 17:40:11, Thomas Greiner wrote: > https://codereview.adblockplus.org/29339552/diff/29339553/skin/options.css > File skin/options.css (right): > > https://codereview.adblockplus.org/29339552/diff/29339553/skin/options.css#newcode1203 ...
April 8, 2016, 3:56 p.m. (2016-04-08 15:56:05 UTC) #3
Thomas Greiner
April 15, 2016, 1:24 p.m. (2016-04-15 13:24:22 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld