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

Commit

Permalink
Merge pull request #6 from jostw/josyeh/Marketplace/master/Bug1228204
Browse files Browse the repository at this point in the history
Bug 1228204 - [TV][2.5] Press option on app's icon will inform system app of which app is focuesd
  • Loading branch information
jostw committed Dec 11, 2015
2 parents 0d109fe + f59571e commit 2d967d4
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 1 deletion.
6 changes: 6 additions & 0 deletions src/media/js/views/homepage.js
Expand Up @@ -83,6 +83,12 @@ define('views/homepage',
});
});

z.page.on('contextmenu', '.focusable', function() {
if (!caps.firefoxOS) {
return false;
}
});

return function(builder) {
builder.start('homepage.html');

Expand Down
8 changes: 7 additions & 1 deletion src/templates/homepage.html
Expand Up @@ -4,10 +4,16 @@
{% defer (url=api('apps'), pluck='objects', as='apps', key='slug') %}
<ul class="clearfix">
{% for app in this %}
<li class="app-list-app focusable" data-slug="{{ app.slug }}">
<li class="app-list-app focusable" data-slug="{{ app.slug }}"
contextmenu="contextmenu-{{ loop.index }}">
<img src="{{ app.icons['128'] }}"
alt="{{ app.name }}" title="{{ app.name }}">
<span class="name">{{ app.name }}</span>

<menu type="context" id="contextmenu-{{ loop.index }}">
<menuitem label="{{ app.manifest_url }}"></menuitem>
</menu>

</li>
{% endfor %}
</ul>
Expand Down

0 comments on commit 2d967d4

Please sign in to comment.