Closed
Bug 887555
Opened 11 years ago
Closed 11 years ago
[Leo][Here Maps][About][Terms] User needs to zoom in when tapping on Advertisements to read context
Categories
(Tech Evangelism Graveyard :: Preinstalled B2G Apps, defect, P4)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: jcouassi, Assigned: jussi.ar.silfver)
References
Details
Attachments
(4 files)
Zoom-in is needed to read context in Advertisement
Repro Steps:
1) Updated to Leo Build ID: 20130620160852
2) Open Here Maps
3) Click on Option button (Top Left Corner)
4) Tap on 'About'
5) Tap on 'Advertisements'
6) View what is listed
Actual:
When clicking on Advertisements user has to zoom in to read context
Expected:
Context is eligible and able to read without using the zoom in method
Environmental Variables
Gecko: /rev/
Gaia: ef85283d8a041cecf9a37348cf6a0b580804f3d6
Platform Version: 18.1
Notes:
Issue also occurs in Personal Data page
Repro frequency: 100%
Comment 1•11 years ago
|
||
I have a problem to reproduce this issue. What do you mean by "Advertisements" on "About" page? Could you attach a screenshot and add the version number of HERE Maps?
(In reply to pawel.wojciechowski from comment #1)
> I have a problem to reproduce this issue. What do you mean by
> "Advertisements" on "About" page? Could you attach a screenshot and add the
> version number of HERE Maps?
I am sorry I forgot a step.....
Repro Steps:
1) Updated to Leo Build ID: 20130620160852
2) Open Here Maps
3) Click on Option button (Top Left Corner)
4) Tap on 'About'
5) Tap on 'Advertisements'
6) Tap on 'Nokia Service Terms'
7) View what is listed
Screen shots of what you see are below.
Comment 6•11 years ago
|
||
Thank you Jeni, now I can reproduce it. The order of repro steps is slightly different:
2) Open Here Maps
3) Click on Option button (Top Right Corner)
4) Tap on 'About'
5) Tap on 'Nokia Service Terms' (or 'Service Terms' in HERE Maps version 1.8.67)
6) Tap on 'Advertisements'
7) View what is listed
Issue repros on v08m
Leo Build ID: 20130721095109
Gecko: /rev/
Gaia: 47872e74e310a8b575ab39f1a9f334e229be995c
Platform Version: 18.1
When user clicks on 'Advertisements' user has to scroll in view what is listed
Comment 8•11 years ago
|
||
what is the status of this bug?
Comment 9•11 years ago
|
||
I was able to reproduce this on build 18.1 20130921041201.
Pawel, any update on this getting fixed?
Flags: needinfo?(pawel.wojciechowski)
Comment 10•11 years ago
|
||
Hi Donovan,
This issue is not fixed yet but we will take care of it within the next couple of weeks. If you want to raise a priority of this issue, please let us know.
Cheers,
Pawel
Flags: needinfo?(pawel.wojciechowski)
Comment 11•11 years ago
|
||
(In reply to Pawel Wojciechowski from comment #10)
> This issue is not fixed yet but we will take care of it within the next
> couple of weeks. If you want to raise a priority of this issue, please let
> us know.
Thank you. I don't think it is a huge priority but just wanted to get a status update.
Comment 12•11 years ago
|
||
Please provide a resolution or a date for resolution of this bug. It has been open for 3+ months.
Flags: needinfo?(andy.tjin)
Priority: -- → P2
Comment 13•11 years ago
|
||
I have put some pressure on the team that maintains the mobi pages. I hope this will be resolved soon.
Flags: needinfo?(andy.tjin)
Comment 14•11 years ago
|
||
The page is showing exactly correct on the Firefox OS Simulator. It is extremely difficult for that team to debug if they don't have a device. Could anyone please investigate why this particular page, which is using the same template as the other Service Terms, would be rendered differently?
Comment 15•11 years ago
|
||
I will look into it.
Updated•11 years ago
|
Assignee: nobody → dpreston
Severity: normal → minor
Status: NEW → ASSIGNED
Priority: P2 → P3
Comment 16•11 years ago
|
||
Reproduced this on a Keon device (OS v1.1) with the new HERE Maps 2.0
Comment 17•11 years ago
|
||
I'm still investigating.
Comment 18•11 years ago
|
||
Example of working page, terms.html
Comment 19•11 years ago
|
||
Example of broken page, ads.html
Comment 20•11 years ago
|
||
I don't think we can call this a Tech Evangelism bug.
I have attached two files, terms.html and ads.html. These files are saved from the same files served by the maps app. The only difference is that I inserted newlines between dom elements to make diffing easier.
If you diff the two files, you will see that they are identical except for the content. The css stylesheet, a css reset, is identical. The structure of the dom is identical. The only difference is the text included in the two pages.
Why is this weird? Because terms.html displays fine, and ads.html is broken. If there is something Nokia can do to fix ads.html, I don't know what it is.
Another interesting data point is that the ads.html page is only broken on the phone; in the simulator, it renders just fine.
I think this is a platform bug.
Updated•11 years ago
|
Assignee: dpreston → nobody
Component: Preinstalled B2G Apps → General
Product: Tech Evangelism → Firefox OS
Comment 21•11 years ago
|
||
Hey Donovan,
Since it's a platform bug--do you think we should redirect the bug to Jason Smith or should I create a new bug?
Flags: needinfo?(dpreston)
Comment 22•11 years ago
|
||
I'll just mark it 1.3? to see what triage for 1.3 thinks about it.
blocking-b2g: --- → 1.3?
Flags: needinfo?(dpreston)
Updated•11 years ago
|
Component: General → Panning and Zooming
Product: Firefox OS → Core
Comment 23•11 years ago
|
||
Milan
Please review if this bug is a valid one given all the changes in panning and zooming from Graphics team.
Flags: needinfo?(milan)
Comment 24•11 years ago
|
||
I had trouble reproducing this, but that could have been a device. For those that can reproduce it, does setting APZC on (from the power menu) and restarting the phone still show the problem?
Comment 25•11 years ago
|
||
OK, I can reproduce this with attachments. As mentioned in comment 20, terms.html and ads.html only differ by the content inside of <p> ... </p> tag. Playing with it a bit, I can get the bug to show up or not when the content differs by a single character. Content of <= 230 characters demonstrates this problem, and when we have > 230 characters, the problem goes away.
Taking ads.html and adding:
to the content of the <p> tag makes the problem go away. This content fails:
123456789 123456789 123456789 123456789 123456789 123456789 123456789 1\
23456789 123456789 123456789 123456789 123456789 123456789 123456789 123456789 \
123456789 123456789 123456789 123456789 123456789 123456789 123456789 123456789
this content succeeds (just an extra X at the start.)
X 123456789 123456789 123456789 123456789 123456789 123456789 123456789 1\
23456789 123456789 123456789 123456789 123456789 123456789 123456789 123456789 \
123456789 123456789 123456789 123456789 123456789 123456789 123456789 123456789
-------------
David, is this somehow layout? Or is CSS telling us exactly to behave the way we do when we have less than certain amount of text?
Component: Panning and Zooming → Layout
Flags: needinfo?(milan) → needinfo?(dbaron)
Comment 26•11 years ago
|
||
This is likely the result of our font inflation code. Small pieces of text don't get inflated the same way longer pieces of text do.
Comment 27•11 years ago
|
||
Fundamentally, I think pages like these should be using a <meta viewport> to indicate that they don't need (and in fact don't want) a full desktop viewport that needs to be panned and zoomed around in (which is our default assumption for Web content, since there's a lot of content designed for desktop that people want to use on mobile). People designing content like this that's designed for mobile should use <meta viewport> appropriately. I think that's a much more appropriate fix than trying to fiddle with the font inflation heuristics more (which might be worth doing, but not for a single page).
Flags: needinfo?(dbaron)
Comment 28•11 years ago
|
||
(I think it's worth opening a separate bug on why the simulator isn't showing the same font inflation behavior as the device... though it might be worth testing the simulator with different device widths since some of the heuristics will change as a function of device width.)
Back to tech evangelism, since stuff in preinstalled B2G apps should be using <meta viewport> rather than relying on full desktop-to-mobile conversion heuristics.
Assignee: nobody → dpreston
Component: Layout → Preinstalled B2G Apps
Product: Core → Tech Evangelism
Comment 29•11 years ago
|
||
Ok, I think the <meta viewport> suggestion is a great one. Thanks a lot, David.
Updated•11 years ago
|
blocking-b2g: 1.3? → ---
Comment 30•11 years ago
|
||
Pawel, can you see if the <meta viewport> suggestion helps here? (Comment 28)
If you are not the right person to look into this, can you bring it to the attention of the right person?
Thanks
Flags: needinfo?(pawel.wojciechowski)
Comment 31•11 years ago
|
||
Hi Donovan, thanks for your suggestion, I'll verify if this solution works fine.
Flags: needinfo?(pawel.wojciechowski)
Comment 32•11 years ago
|
||
What's the status? Do we have a potential fix date?
Assignee | ||
Comment 33•11 years ago
|
||
Hello all,
This issue is in our backlog but unfortunately it has been deprioritized.
Sorry for the inconvenience.
- Jussi -
Comment 34•11 years ago
|
||
(In reply to jussi.ar.silfver@here.com from comment #33)
> Hello all,
>
> This issue is in our backlog but unfortunately it has been deprioritized.
> Sorry for the inconvenience.
>
> - Jussi -
This and other issues are affecting pre-loads on all devices in market. The carriers are keen to have these resolved.
Updated•11 years ago
|
Assignee: dpreston → jussi.ar.silfver
Priority: P3 → P4
Comment 35•11 years ago
|
||
Was unable to reproduce on ZTE V790 v1.3 Changing to Resolved - Works for me.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → WORKSFORME
Updated•6 years ago
|
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•