gasolin@mozilla.com Assignee
	Description • 11 years ago

some device did not have telephony support.

We'd hide dialer app in homescreen and settings/permission list if no telephony support

	gasolin@mozilla.com Assignee
	Comment 1 • 11 years ago

@crdlc do you know how FTU hide itself from homescreen? maybe we can do something like that for dialer when telephony is not available.

	Cristian Rodriguez (:crdlc)
	Comment 2 • 11 years ago

Some apps are not rendered for the homescreen depending on role

https://github.com/mozilla-b2g/gaia/blob/master/apps/homescreen/js/grid.js#L15

Good question, why does not homescreen paint FTU app?

I didn't that but looking an answer -> https://github.com/mozilla-b2g/gaia/blob/master/apps/homescreen/js/grid.js#L1004

Entry points without icons defined in the manifest are hidden :)

	Cristian Rodriguez (:crdlc)
	Comment 3 • 11 years ago

Sorry, did I change the koi flag? how?

	gasolin@mozilla.com Assignee
	Comment 4 • 11 years ago

@crdlc, per discussion with george, do you mean add a blacklist in config options(customization) to filter out dialer?

	Cristian Rodriguez (:crdlc)
	Comment 5 • 11 years ago

I gonna add to Vivien here because he was following the last change about roles. I agree that entry points without icons have to be hidden is a hack and I don't like that but I prefer that Vivien'll expose his opinion. With Vivien's opinion, you should continue with this hack or adding a blacklist

	Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)
	Comment 6 • 11 years ago

(In reply to Cristian Rodriguez (:crdlc) from comment #5)
> I gonna add to Vivien here because he was following the last change about
> roles. I agree that entry points without icons have to be hidden is a hack
> and I don't like that but I prefer that Vivien'll expose his opinion. With
> Vivien's opinion, you should continue with this hack or adding a blacklist

Both ways are wrong imo but we need to do something for now so pick what you like  (i personally prefer not showing the icon on the homescreen if the entry point does not have an icon) :)

The longer term plan is to get rid of entry points if possible so that will be resolve but there are probably a few months of work to achieve this.

	Cristian Rodriguez (:crdlc)
	Comment 7 • 11 years ago

I really think so, the simpler the better right now

	Francisco Jordano [:arcturus] [:francisco]
	Comment 8 • 11 years ago

+10000 to remove entry points

;)

	gasolin@mozilla.com Assignee
	Comment 9 • 11 years ago

add `app.launch_path.blacklist` settings attribute to filter out entrypoint from homescreen

will add another PR to hide icon in settings/permission list

	gasolin@mozilla.com Assignee
	Comment 10 • 11 years ago

In settings/app permissions there's no communication apps, so the last PR is sufficient

	Cristian Rodriguez (:crdlc)
	Comment 11 • 11 years ago

Comment on attachment 805142 [details]
pull request redirect to github

Please Vivien review it because I prefer that this kind of decisions will be taken by you. IMHO you have more knowledge about settings... Thanks

	Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)
	Comment 12 • 11 years ago

Comment on attachment 805142 [details]
pull request redirect to github

Hey Cristian, even if I don't like the way it is done here there is not any other possibilities right now. The way to solve that is a long term plan that will allow us to get rid of |entry_points| entirely.

So the feature in this patch, even if it is a hack, is fine.

	Cristian Rodriguez (:crdlc)
	Comment 13 • 11 years ago

Please review my comment on Github

	gasolin@mozilla.com Assignee
	Comment 14 • 11 years ago

I've revised the PR. Please kindly review it again, thanks.

	Cristian Rodriguez (:crdlc)
	Comment 15 • 11 years ago

Comment on attachment 805142 [details]
pull request redirect to github

I have just a question in Github, good work as usual!. Although we don't have best approach thought, I don't like to wait for mozSettings in order to paint the grid. But your implementation is totally perfect

Thanks a lot Fred

	gasolin@mozilla.com Assignee
	Comment 16 • 11 years ago

merged to gaia-master https://github.com/mozilla-b2g/gaia/commit/2c3dc472094a278da97d43e7cfe0b6d2a075b24c thanks!

	gasolin@mozilla.com Assignee
	Comment 17 • 11 years ago

uplift to v1.2 https://github.com/mozilla-b2g/gaia/commit/9b1b262e8fde58be453fb05ed91c0e93ab86d394

	Kevin Grandon :kgrandon
	Comment 18 • 11 years ago

Fred, we had to back this out due to contact app launching breaking. Please add a unit test for this case if possible. Reopening this bug.

Backed out of master: https://github.com/mozilla-b2g/gaia/commit/53f63e4a5e1ca6a650b9b1dce8039c79741b72a7
Backed out of v1.2: https://github.com/mozilla-b2g/gaia/commit/c932c482c6944fa32724ce7af9e5423c4c2bcccd

	gasolin@mozilla.com Assignee
	Comment 19 • 11 years ago

Kevin, I found the cause and will add unit test for entrypoints, thanks

	gasolin@mozilla.com Assignee
	Comment 20 • 11 years ago

the root cause is I miss used 'return' instead of 'continue' in loop, which skip the following entrypoint process (that effect all entrypoint path but not only contacts app)

Sad I can't find a good way to write testcase for it.

	gasolin@mozilla.com Assignee
	Comment 21 • 11 years ago

@crdlc I've updated the source, please kindly review it again, thanks

	Cristian Rodriguez (:crdlc)
	Comment 22 • 11 years ago

Comment on attachment 808477 [details]
2nd pull request

Good job! Please address my last two comments in Github and merge it. Thanks a lot

	gasolin@mozilla.com Assignee
	Comment 23 • 11 years ago

merged in gaia-master https://github.com/mozilla-b2g/gaia/commit/1c6db3efc258b3afdf3cf25079a6799d06c0adba
uplift v1.2 https://github.com/mozilla-b2g/gaia/commit/54c0b89a1ac81e8f1e40ff76aefe9dbb722a1b3b

Thanks!

	gasolin@mozilla.com Assignee
	Comment 24 • 11 years ago

viral, 

per offline discussion, in this bug we use blacklist filter instead of feature detection with mozTelephony API, so we could hide dialer even if some devices comes with only DATA plan but without phone feature.

Is this what you and Bruce concern about?

	viral [:viralwang]
	Comment 25 • 11 years ago

I think it's ok to use blacklist in this case to hide dialer.
For data access only but without phone feature, it won't need dialer also!

what we worried about is the cases like 'no sim card' (the items in settings), is that proper to use blacklist to filter also?

	gasolin@mozilla.com Assignee
	Comment 26 • 11 years ago

viral, for 'no sim card' case, we'd hide them via feature detection if mozConnection API is not exist.
see bug 911676