651266 - Google Map Maker is missing elements

Status: RESOLVED FIXED

(Tech Evangelism Graveyard :: English US, defect)

Description

[Leman Bennett [Omega]]

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0a1) Gecko/20110419 Firefox/6.0a1
Build Identifier: 

The Google Map Maker is missing several styling elements that break its usability. The form is correctly styled in Fx4, but not in the latest nightly.

Reproducible: Always

Steps to Reproduce:
1. Load Mapmaker 
2. Find a point and edit it.
Actual Results:  
Various elements including the data form is missing its styles.

Expected Results:  
Styled form according to Fx4, Chrome, Opera, IE9.

I tried spoofing the UA, But that didn't seem to work.

Comment 1

[Alice0775 White]

Regression window(m-c hourly):
Works:
http://hg.mozilla.org/mozilla-central/rev/048b078d112d
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.2a1pre) Gecko/20110323 Firefox/4.2a1pre ID:20110323121728
Fails:
http://hg.mozilla.org/mozilla-central/rev/6eb93bfd55b7
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.2a1pre) Gecko/20110323 Firefox/4.2a1pre ID:20110323124013
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=048b078d112d&tochange=6eb93bfd55b7

Comment 2

[Alice0775 White]

Triggered by 
52e34cbb1d4e	Boris Zbarsky — Bug 616137. Skip to the nearest ']' when parsing an attr selector fails. r=dbaron

Comment 3

[Alice0775 White]

Attachment: responseed css

http://www.google.com/mapmaker/v=ef9b9f060e987b6a/gw.css?hl=en

There is a typo "[" in gw.css.
.gw-review-internal [font-size:x-small;padding-bottom: 0.3em;text-align: right;}

So, TE?

Comment 4

[Boris Zbarsky [:bzbarsky]]

Yep.  I'll contact the google folks.

Comment 5

[JP Rosevear [:jpr]]

Boris, can you ping Google about this again?

Comment 6

[Boris Zbarsky [:bzbarsky]]

Just did, via our normal contact channel, but maybe we should make use of other contacts too?

Comment 7

[Christopher Blizzard (:blizzard)]

I got mail from Michael Davidson.  He says that this has been fixed internally and will be fixed when they do another mapmaker push.  (For google people, the internal reference is s/20902274.)

Keeping the bug open until the push happens.

Comment 8

[Christopher Blizzard (:blizzard)]

Google says it's already pushed and I verified the functionality and the source file.  Looks good!