Closed
Bug 1100127
Opened 10 years ago
Closed 10 years ago
Regression rendering ownCloud News (Fx 33 to 34)
Categories
(Web Compatibility :: Desktop, defect)
Web Compatibility
Desktop
Tracking
(firefox33 unaffected, firefox34+ wontfix)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox33 | --- | unaffected |
firefox34 | + | wontfix |
People
(Reporter: micmon, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
image/png
|
Details |
The attached screenshot shows a regression in how Firefox displays the ownCloud News component between Firefox 33 and 34:
- Top: broken (Firefox 34)
- Bottom: correct (Firefox 33)
Comment 1•10 years ago
|
||
Got a link to the site in question?
Reporter | ||
Comment 2•10 years ago
|
||
(In reply to Josh Matthews [:jdm] from comment #1)
> Got a link to the site in question?
ownCloud [1] is a self-hosted cloud supporting various "apps" (like plugins). One of those is the news app [2] which shows the regression.
If you do not have access to an ownCloud instance I could create a test account for you on my installation or try to come up with a somewhat reduced test page.
[1] http://owncloud.org/
[2] https://github.com/owncloud/news
Comment 3•10 years ago
|
||
The reduced test page would be ideal, if possible. You could also help us narrow down the regression range window using the mozregression tool - http://mozilla.github.io/mozregression/
Reporter | ||
Comment 4•10 years ago
|
||
(In reply to Josh Matthews [:jdm] from comment #3)
> The reduced test page would be ideal, if possible.
I will try.
Comment 6•10 years ago
|
||
Tracking 34. Note that we are very late in the 34 cycle. If we're going to take a fix, we're going to need it to be ready by Thu, Nov 20.
Reporter | ||
Comment 7•10 years ago
|
||
Unfortunately I was not able to produce a meaningful reduced testcase.
Comment 8•10 years ago
|
||
Access to an ownCloud instance would be pretty helpful, then....
Reporter | ||
Comment 9•10 years ago
|
||
(In reply to Please do not ask for reviews for a bit [:bz] from comment #8)
> Access to an ownCloud instance would be pretty helpful, then....
I will send you url/username/pass by email.
Comment 10•10 years ago
|
||
Thank you.
Looks like the behavior changed in this m-c push range: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=63f44b4968c2&tochange=82df3654cd80
This is the range in which bug 1015474 landed.
Looking at the page, it is in fact using "display: flex" for "#app-content .utils ul" and if I add this CSS rule to the page:
#app-content .utils ul * { min-width: 0 }
the layout goes back to what it used to be.
So this looks like the page depending on the old spec draft behavior, unfortunately. I filed https://github.com/owncloud/news/issues/661
Blocks: 1015474, minsizeauto-fallout
Updated•10 years ago
|
See Also: → https://github.com/owncloud/news/issues/661
This is just a bit too late to take a fix for 34. Though it looks super promising that we have progress!
status-firefox35:
--- → ?
Comment 12•10 years ago
|
||
I don't think there's anything to fix on our end here, if my analysis is correct. We're just following the (updated) spec.
Comment 13•10 years ago
|
||
Yeah, based on comment 10, this turned out to be a Tech Evang bug. (& the actual fixing is now mostly tracked in the github issue for owncloud)
(I thought I already reclassified as tech evang, but I must've not hit "submit" -- sorry about that. Reclassifiying & canceling tracking-firefox35 request.)
status-firefox35:
? → ---
Component: General → Desktop
OS: Linux → All
Product: Core → Tech Evangelism
Version: 34 Branch → unspecified
Reporter | ||
Comment 14•10 years ago
|
||
Thanks to everyone evolved. Hopefully this will be fixed in OC 7.0.4 so that it will work fine in the next stable Firefox.
Comment 15•10 years ago
|
||
Fix landed, on the ownCloud issue:
https://github.com/owncloud/news/commit/dfeb06fea38251b2e34a2ef0e0d586ddd1d7d630
Marking this as FIXED.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•6 years ago
|
Product: Tech Evangelism → Web Compatibility
You need to log in
before you can comment on or make changes to this bug.
Description
•