829825 - Browser causes space between td's after default set to zero

Status: VERIFIED FIXED

(Core :: SVG, defect)

Description

[RDall]

Attachment: td border Bug See Red Highlight

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:18.0) Gecko/20100101 Firefox/18.0
Build ID: 20130104151948

Steps to reproduce:

Viewed TD with no padding no border and showed white border around images in a td. 


Actual results:

It showed white border between images with td. See Screenshot See red highlighted region http://cl.ly/image/123h0L0M0L12 But the white space flickers on and off when you re-size the browser window. See dev site and scroll to Meet our Surgeons to see issue. http://dal.32spokes.com/


Expected results:

No spaces between the tds. See screenshot http://cl.ly/image/0K0W233B0Q3o. I have also checked this on Chrome, IE and no issue appears at all.

Comment 1

[Kevin Brosnan [Ex-Mozilla]]

Please provide code, ideally a minimized test case that shows the issue.

Comment 2

[RDall]

(In reply to Kevin Brosnan [:kbrosnan] from comment #1)
> Please provide code, ideally a minimized test case that shows the issue.

I have already provided a page where you can see the issue. 

http://dal.32spokes.com/

You can see all the CSS and HTML code I have used from there. This only happens in the FF browser. Not sure why I would need to take the time out to replicate the issue on another page when it only affects FF browser.

Comment 3

[(mostly gone) XtC4UaLL [:xtc4uall]]

This is a Regression between (at least) Firefox 14 vs 17.

Comment 4

[Alice0775 White]

Regression window(m-c)
Good:
http://hg.mozilla.org/mozilla-central/rev/6dae57cd2f85
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/17.0 Firefox/17.0 ID:20120720145842
Bad:
http://hg.mozilla.org/mozilla-central/rev/045c11dd41a6
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/17.0 Firefox/17.0 ID:20120720205642
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=6dae57cd2f85&tochange=045c11dd41a6

Regression window(m-i)
Good:
http://hg.mozilla.org/integration/mozilla-inbound/rev/a746aaa32b22
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/17.0 Firefox/17.0 ID:20120720114508
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/62f19ed60528
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/17.0 Firefox/17.0 ID:20120720115241
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=a746aaa32b22&tochange=62f19ed60528

Suspected; Bug 614723

Comment 5

[(mostly gone) XtC4UaLL [:xtc4uall]]

If you disable

#surgeon-rl td a:link img.desaturate {
    filter: url("inc/desaturate.svg#greyscale");
}

in the Inspector the Issue of the White Gap is gone.

Comment 6

[Alice0775 White]

And the gap depend on browser width.

Comment 7

[Alex Keybl [:akeybl]]

We'd accept a low risk fix here, but do not feel the need to track for upcoming releases given this is a regression in FF17 (unreported until 1.5 months after its release).

Comment 8

[RDall]

Hi… I filed this report as I didn't see it disappear on the latest update… Also I tend to think if your not part of the solution your part of the problem… 

Also if the client mentions that they see the issue I can verify it isn't my HTML and CSS but the browser itself.

I hope to see it fixed at some point… (It would be over my head and knowledge as to how) But I do agree it isn't a huge priority.

Comment 9

[Jonathan Watt [:jwatt]]

Thanks for reporting the issue, RDall.

Comment 10

[Daniel Holbert [:dholbert]]

[adding "testcase-wanted" -- ideally it'd be great to have a testcase attached to this bug page, so that we can still test this in case the remote site ends up changing at some point]

Comment 11

[Alice0775 White]

Attachment: reduced html zipfile

Comment 12

[Daniel Holbert [:dholbert]]

It looks like the gap appears at alternate pixels of browser width.
No gap when browser is N pixels wide; gap when it's N+1 pixels; no gap at N+2; gap at N+3, etc

Comment 13

[Jonathan Watt [:jwatt]]

A 260.99 KB zip file isn't my idea of a reduced testcase. ;) I don't have time to reduce it right now, but if someone else does that and attaches a (say, less than 1 KB unzipped) testcase, then I'm much more likely to be able to take a look at the issue.

Comment 14

[RDall]

I am a little surprised this is still open after almost 9 months.  Can we get a refresh on this ticket???

Comment 15

[Ryan VanderMeulen [:RyanVM]]

If you want to help, comment 13 suggests that a more reduced testcase would be appreciated.

Comment 16

[Alice0775 White]

(In reply to Jonathan Watt [:jwatt] from comment #13)
> A 260.99 KB zip file isn't my idea of a reduced testcase. ;) I don't have
> time to reduce it right now, but if someone else does that and attaches a
> (say, less than 1 KB unzipped) testcase, then I'm much more likely to be
> able to take a look at the issue.

???

Comment 17

[RDall]

Test case size seems fine to me… As I have said previously fixing the issue is well over my head as I am just a web designer… 

But I was going through an old email and I found the bug ticket so I though I would check the status and saw it wasn't closed or even assigned.

Comment 18

[Jonathan Watt [:jwatt]]

This seems to be WORKSFORME now on Nightly.

Comment 19

[(mostly gone) XtC4UaLL [:xtc4uall]]

(In reply to Jonathan Watt [:jwatt] from comment #18)
> This seems to be WORKSFORME now on Nightly.

Confirmed. Also WFM against Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20100101 Firefox/25.0 ID:20130831004004 CSet: 9dd5f0e7f716

Comment 20

[Alice0775 White]

Attachment: reduced 810Byte

Comment 21

[Alice0775 White]

I cannot reproduce in Aurora25.0a2 and later.

Fixed window(m-i)
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/b4402b6dba21
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130724 Firefox/25.0 ID:20130724055940
Fixed:
http://hg.mozilla.org/integration/mozilla-inbound/rev/74633f25ca05
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130724 Firefox/25.0 ID:20130724060938
Pushlg:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=b4402b6dba21&tochange=74633f25ca05

Fixed by:
	517eaa8ba87e	Xidorn Quan — Bug 786254 - SVG effects rendering doesn't pixel snap, causing faint vertical lines appear on some tabs. r=roc

Comment 22

[Jonathan Watt [:jwatt]]

Thanks, Alice.

Comment 23

[Ryan VanderMeulen [:RyanVM]]

Probably late in the game to request uplift to beta for Fx24, but you can always ask in bug 786254. Looks like the patch is pretty simple and comes with a test.

Comment 24

[Xidorn Quan [:xidorn] UTC+11]

(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #23)
> Probably late in the game to request uplift to beta for Fx24, but you can
> always ask in bug 786254. Looks like the patch is pretty simple and comes
> with a test.

How can I request the uplift? I really hope the bug can be fixed in release version ASAP since it affects my everyday use.

Comment 25

[Ryan VanderMeulen [:RyanVM]]

Request beta approval on the patch in the bug and answer the questions it asks you.

Comment 26

[bhavana bajaj [:bajaj]]

This should be resolved starting firefox 25 hence wontfixing Fx24 given we could not uplift https://bugzilla.mozilla.org/show_bug.cgi?id=786254 in our last two beta's left on Fx24.

Comment 27

[Petruta Horea [:phorea], Desktop QA]

Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20100101 Firefox/25.0
Mozilla/5.0 (X11; Linux i686; rv:25.0) Gecko/20100101 Firefox/25.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:25.0) Gecko/20100101 Firefox/25.0

Verified as fixed on Firefox 25 beta 1 (20130917123208).

Comment 28

[Petruta Horea [:phorea], Desktop QA]

Mozilla/5.0 (X11; Linux x86_64; rv:26.0) Gecko/20100101 Firefox/26.0
Mozilla/5.0 (Windows NT 6.0; rv:26.0) Gecko/20100101 Firefox/26.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:26.0) Gecko/20100101 Firefox/26.0

Verified as fixed on latest Aurora 26.0a2 (buildID: 20131003004003).