Closed
Bug 1500530
Opened 6 years ago
Closed 6 years ago
<input type="file"> label width is too small. (tori.fi picture upload broken in Nightly)
Categories
(Core :: Layout: Form Controls, defect)
Tracking
()
VERIFIED
FIXED
mozilla65
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox63 | --- | unaffected |
firefox64 | + | verified |
firefox65 | + | verified |
People
(Reporter: hsivonen, Assigned: MatsPalmgren_bugz)
References
()
Details
(Keywords: regression)
Attachments
(3 files)
(deleted),
text/html
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
patch
|
jfkthame
:
review+
RyanVM
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
Steps to reproduce:
1) Navigate to https://www2.tori.fi/ai/form/0?ca=18
2) Towards to the bottom of the page, locate the square with the text "Lisää kuva"
4) Move the mouse cursor over the text "Lisää kuva" so that it changes color
3) Click the text "Lisää kuva"
Actual results:
Nothing
Expected results:
Expected a file upload file picker to show up.
Additional info:
This works on the release channel and broke on Nightly in the last 6 days.
Comment 1•6 years ago
|
||
I repteated the bisection twice and always got bug 1495261 as as result
6:24.38 INFO: Last good revision: 4c10bee140e0e63deeb9bb9ea646dbbb2b7668d6
6:24.38 INFO: First bad revision: db73000c424ba29b41f6f5438ce6a389b4e529ef
6:24.38 INFO: Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=4c10bee140e0e63deeb9bb9ea646dbbb2b7668d6tochange=db73000c424ba29b41f6f5438ce6a389b4e529ef
Blocks: 1495261
Reporter | ||
Comment 3•6 years ago
|
||
For context, this site is relatively notable in Finland. (It's a popular site for posting items for second-hand sale.)
Updated•6 years ago
|
Component: General → File Handling
Product: Core → Firefox
Comment 4•6 years ago
|
||
I'm able to reproduce the problem on 4c10bee140e0e63deeb9bb9ea646dbbb2b7668d6, before bug 1495261 landed. Bug 1495261 should not affect this page, or any other one; it changes some JS that only runs when recording a tab using Web Replay.
Flags: needinfo?(bhackett1024)
Updated•6 years ago
|
Keywords: regressionwindow-wanted
Comment 5•6 years ago
|
||
[Tracking Requested - why for this release]: Stops working <Input type=file> in some cases due to regression
Regression window:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=90853a59691e0c77014c38049bfb38b2b6ca1e16&tochange=4c10bee140e0e63deeb9bb9ea646dbbb2b7668d6
Regressed by: Bug 1495153
:mats, Could you please look into this?
status-firefox63:
--- → unaffected
status-firefox64:
--- → affected
status-firefox65:
--- → affected
status-firefox-esr60:
--- → unaffected
tracking-firefox64:
--- → ?
Component: File Handling → Layout: Form Controls
Flags: needinfo?(mats)
Keywords: regressionwindow-wanted → regression
Product: Firefox → Core
Version: unspecified → 64 Branch
Assignee | ||
Comment 7•6 years ago
|
||
It seems the intrinsic size has changed. I'm guessing there's
a built-in min-inline-size associated with a XUL label that's
now missing and makes the control smaller than before. I haven't
found where that min-inline-size comes from yet though...
Assignee | ||
Comment 8•6 years ago
|
||
This rule should apply and is the same size as before:
https://searchfox.org/mozilla-central/rev/a7f4d3ba4fbfe3efbde832869f1d672fce7122f6/layout/style/res/forms.css#493-494
so I'm not sure why there's a difference, hmm...
Assignee | ||
Comment 9•6 years ago
|
||
Oh darn, our HTML <label> is an inline by default so that declaration
simply doesn't apply... LOL.
Comment 10•6 years ago
|
||
(In reply to Mats Palmgren (:mats) from comment #7)
> Created attachment 9019746 [details]
> Testcase
>
> It seems the intrinsic size has changed. I'm guessing there's
> a built-in min-inline-size associated with a XUL label that's
> now missing and makes the control smaller than before. I haven't
> found where that min-inline-size comes from yet though...
And field becomes too short when choose file with short file name. This is unacceptable bug.
Updated•6 years ago
|
Summary: tori.fi picture upload broken in Nightly → <input type="file"> label width is too small. (tori.fi picture upload broken in Nightly)
Assignee | ||
Comment 11•6 years ago
|
||
Attachment #9019903 -
Flags: review?(jfkthame)
Updated•6 years ago
|
Attachment #9019903 -
Flags: review?(jfkthame) → review+
Comment 12•6 years ago
|
||
Pushed by mpalmgren@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4896323e685b
Make the anonymous child <label> have display:inline-block so that it applies its min-inline-size. r=jfkthame
Comment 13•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Assignee | ||
Comment 14•6 years ago
|
||
Comment on attachment 9019903 [details] [diff] [review]
Make the anonymous child <label> have display:inline-block so that it applies its min-inline-size
[Beta/Release Uplift Approval Request]
Feature/Bug causing the regression: Bug 1495153
User impact if declined:
Is this code covered by automated tests?: Yes
Has the fix been verified in Nightly?: No
Needs manual test from QE?: No
If yes, steps to reproduce:
List of other uplifts needed: None
Risk to taking this patch: Medium
Why is the change risky/not risky? (and alternatives if risky):
String changes made/needed:
Attachment #9019903 -
Flags: approval-mozilla-beta?
Updated•6 years ago
|
Comment 15•6 years ago
|
||
Comment on attachment 9019903 [details] [diff] [review]
Make the anonymous child <label> have display:inline-block so that it applies its min-inline-size
[Triage Comment]
Fixes a new visual regression in Fx64 and includes a new reftest. Approved for 64.0b5.
Attachment #9019903 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 16•6 years ago
|
||
bugherder uplift |
Comment 17•6 years ago
|
||
Hi, I reproduced this issue in Firefox Beta 64.0b4 but I can no longer reproduce this issue in Firefox 64.0b5 (https://tools.taskcluster.net/index/gecko.v2.mozilla-beta.latest.firefox/win64-opt) and I can Confirm it as Fixed in 65.0a1 (2018-10-28), I will mark this issue accordingly.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•