DOMUI: Adjust location of "Disable individual plug-ins" link.

DOMUI: Adjust location of "Disable individual plug-ins" link.

BUG=70279
TEST=manual, see bug.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72310

[tfarina, 2011-01-22 18:36:09]

Please, take a look.

[James Hawkins, 2011-01-23 22:14:58]

Mostly LG, but the link needs better padding. It butts up against the 'Manage
Exceptions' button.

[tfarina, 2011-01-24 01:29:12]

On 2011/01/23 22:14:58, James Hawkins wrote:
> Mostly LG, but the link needs better padding. It butts up against the 'Manage
> Exceptions' button.

It has the same look and feel of the other links used in Settings, see
screenshots here: http://imgur.com/atCIg&G6BP0

[James Hawkins, 2011-01-24 01:33:17]

On 2011/01/24 01:29:12, tfarina wrote:
> On 2011/01/23 22:14:58, James Hawkins wrote:
> > Mostly LG, but the link needs better padding. It butts up against the
'Manage
> > Exceptions' button.
> 
> It has the same look and feel of the other links used in Settings, see
> screenshots here: http://imgur.com/atCIg&G6BP0

Correct. Both instances are wrong.

[tfarina, 2011-01-24 02:09:22]

On 2011/01/24 01:33:17, James Hawkins wrote:
> > It has the same look and feel of the other links used in Settings, see
> > screenshots here: http://imgur.com/atCIg&G6BP0
> 
> Correct. Both instances are wrong.

Did some tweaks, please compare the old and the new in the screenshots here:
http://imgur.com/Y8FPD&uzic8

(I didn't upload the new patch yet, waiting to see if it's what you have in
mind).

[James Hawkins, 2011-01-24 02:13:25]

On 2011/01/24 02:09:22, tfarina wrote:
> On 2011/01/24 01:33:17, James Hawkins wrote:
> > > It has the same look and feel of the other links used in Settings, see
> > > screenshots here: http://imgur.com/atCIg&G6BP0
> > 
> > Correct. Both instances are wrong.
> 
> Did some tweaks, please compare the old and the new in the screenshots here:
> http://imgur.com/Y8FPD&uzic8
> 
> (I didn't upload the new patch yet, waiting to see if it's what you have in
> mind).

Uploading would have helped, so I could see what margin you're using. This still
looks like the margin is too small.  5px should be about right.

[tfarina, 2011-01-24 02:27:43]

On 2011/01/24 02:13:25, James Hawkins wrote:
> Uploading would have helped, so I could see what margin you're using. This
still
> looks like the margin is too small.  5px should be about right.

Right, new patch uploaded and screenshot here: http://i.imgur.com/x4Ww1.png

[James Hawkins, 2011-01-24 02:30:21]

On 2011/01/24 02:27:43, tfarina wrote:
> On 2011/01/24 02:13:25, James Hawkins wrote:
> > Uploading would have helped, so I could see what margin you're using. This
> still
> > looks like the margin is too small.  5px should be about right.
> 
> Right, new patch uploaded and screenshot here: http://i.imgur.com/x4Ww1.png

The spacing is the same...make it 10px, then LGTM.

[tfarina, 2011-01-24 02:36:39]

On 2011/01/24 02:30:21, James Hawkins wrote:
> The spacing is the same...make it 10px, then LGTM.

There is a subtle difference, but see the margin with 10px in this screenshot:
http://imgur.com/v1Hq9

Still looks good?

[James Hawkins, 2011-01-24 02:38:43]

On 2011/01/24 02:36:39, tfarina wrote:
> On 2011/01/24 02:30:21, James Hawkins wrote:
> > The spacing is the same...make it 10px, then LGTM.
> 
> There is a subtle difference, but see the margin with 10px in this screenshot:
> http://imgur.com/v1Hq9
> 
> Still looks good?

Sorry, I'm not communicating well enough.  I only want margin-top/bottom, so
try:

margin: 10px 0;

[tfarina, 2011-01-24 02:49:39]

On 2011/01/24 02:38:43, James Hawkins wrote:
> Sorry, I'm not communicating well enough.  I only want margin-top/bottom, so
> try:
> 
> margin: 10px 0;

Done, PTAL! One last screenshot: http://i.imgur.com/RxAyJ.png

[James Hawkins, 2011-01-24 02:50:13]

On 2011/01/24 02:49:39, tfarina wrote:
> On 2011/01/24 02:38:43, James Hawkins wrote:
> > Sorry, I'm not communicating well enough.  I only want margin-top/bottom, so
> > try:
> > 
> > margin: 10px 0;
> 
> Done, PTAL! One last screenshot: http://i.imgur.com/RxAyJ.png

LGTM, thanks.