New issue
Advanced search Search tips

Issue 797084 link

Starred by 1 user
Status: Assigned
Owner:
sgabr...@chromium.org
Cc:
hwi@chromium.org
finnur@chromium.org
tetsui@chromium.org
peter@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

MD notification views don't display contextMessage

Project Member Reported by est...@chromium.org, Dec 21 2017 
The app notification API has something called a "contextMessage", "which is displayed as a 3rd text field in a lighter color font." [1]

Pre-MD notifications respected this (the text "Foobar." in the attached screenshot). MD notifications ignore it. Perhaps this was a design oversight?

[1] https://developer.chrome.com/apps/richNotifications
iZsqsoUKsTM.png
19.2 KB View Download

Comment 1 by est...@chromium.org, Dec 21 2017 

engineer's note: to test this I used

./out/Release/chrome --load-extension=chrome/test/data/extensions/api_test/notifications/api/basic_usage/ --disable-features="NativeNotifications"

after making some minor subtractions to that app's background.js and then launching the notification by calling testBaseFormat() from a developer console.

Comment 2 by sgabr...@chromium.org, Dec 21 2017 

Didn't we choose to remove this from notification because of low usage?

Comment 3 by est...@chromium.org, Dec 21 2017 

It's possible, but it's still supported elsewhere (pre-MD messagecenter notifications and some native platforms). I'm happy to delete it if everyone agrees.

Comment 4 by est...@chromium.org, Feb 12 2018

Cc: hwi@chromium.org
So, what is the verdict?

Comment 5 by sgabr...@chromium.org, Feb 13 2018

Cc: peter@chromium.org finnur@chromium.org
I believe we just discussed this with Peter, Hwi and Finnur and that we were ok proceeding to remove it. 

cc Finnur and Peter to confirm.

Comment 6 by est...@chromium.org, Apr 6 2018 

ping Finnur, Hwi, Peter.^

Comment 7 by finnur@chromium.org, Apr 9 2018 

That is my understanding (that we're deprecating it).
►

Sign in to add a comment