Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Bug 1132996 - fixing the markup order based on presentation order. #28272

Merged
merged 1 commit into from Feb 18, 2015

Conversation

yzen
Copy link
Contributor

@yzen yzen commented Feb 17, 2015

No description provided.

@try-server-hook
Copy link

yzen Yura Zenevich (yzen) started tests. Results

@@ -12,9 +12,8 @@ const LAYOUT_MODE = {

var dom = {};

var ids = ['thumbnail-list-view', 'thumbnails-bottom', 'thumbnail-list-title',
var ids = ['thumbnail-views', 'thumbnails-bottom', 'thumbnail-list-title',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It turns out, the js doesn't use the 'thumbnail-list-view' element so we don't need to add 'thumbnail-views'. Also, 'thumbnails-bottom' is not used so it would be nice if you can remove that as well.

@try-server-hook
Copy link

yzen Yura Zenevich (yzen) started tests. Results

@yzen yzen closed this Feb 18, 2015
@yzen yzen reopened this Feb 18, 2015
@try-server-hook
Copy link

yzen Yura Zenevich (yzen) started tests. Results

@mozilla-autolander-deprecated mozilla-autolander-deprecated merged commit 6d163ab into mozilla-b2g:master Feb 18, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants