Skip to content
This repository has been archived by the owner on Sep 4, 2019. It is now read-only.

As a WebWorks developer, I can enable string pattern matching in my content #112

Open
kwallis opened this issue Jun 6, 2012 · 18 comments
Open

Comments

@kwallis
Copy link
Member

kwallis commented Jun 6, 2012

Acceptance Criteria

  • String pattern matching will be defined as a element in the config.xml, as a child of the element
  • The packager should currently ignore this feature if it is found as a child of an element.
       * This feature should currently be treated as an application-wide feature
       * We should open an issue that this feature does not adhere to our usage of elements, in that it is app wide and not part of the implied "local" scope, and is not applicable inside elements.
  • Use feature id="blackberry.app.stringmatch"
  • If the feature is present, then string pattern matching will be enabled for the application
  • By default, the string matching will be disabled.
@kwallis
Copy link
Member Author

kwallis commented Jul 5, 2012

@nukulb Updated description. Please review

@nukulb
Copy link
Contributor

nukulb commented Jul 26, 2012

@kwallis
thinking blackberry.invoke is a better whitelist namespace for the event since if they listen to event the only reason is to actually invoke some app. So keeping the overall use case in mind I think blackberry.invoke is a better namespace for it.

Also thinking we should do the same thing here as we did for the context menu event. Bring it in later so only deliver the enable disable part for the Beta4 and defer the events to the future.

This task is unblocked now

@kwallis
Copy link
Member Author

kwallis commented Aug 22, 2012

Updated description based on design discussions

@nukulb
Copy link
Contributor

nukulb commented Sep 6, 2012

@kwallis - I think it should be enabled by default?

handing this to a new TWG resource as a ramp up task (its out of priority but its a good ramp up task)

@monicao - welcome !!!

@nukulb
Copy link
Contributor

nukulb commented Sep 6, 2012

maybe a param is better here!! but its fine the way it is I guess

@kwallis
Copy link
Member Author

kwallis commented Sep 6, 2012

I made it disabled because making it a feature, if you include the feature in the config.xml it is basically implicitly saying that it should be enabled. Same argument around removing the "enable" text out of the feature id for flash.

@nukulb
Copy link
Contributor

nukulb commented Sep 6, 2012

just think its better usability as more apps will want it enabled

@kwallis
Copy link
Member Author

kwallis commented Sep 6, 2012

I completely agree. At the same time, it seems backwards to be adding a feature to the config, with a param, only to DISABLE a feature. Right now I am leaning to that being just too weird...

Ken Wallis

Product Manager – BlackBerry WebWorks

Research In Motion

(905) 629-4746 x14369


From: Nukul Bhasin [[email protected]]
Sent: Thursday, September 06, 2012 1:50 PM
To: blackberry/BB10-WebWorks-Framework
Cc: Ken Wallis
Subject: Re: [BB10-WebWorks-Framework] As a WebWorks developer, I can enable string pattern matching in my content (#112)

just think its better usability as more apps will want it enabled


Reply to this email directly or view it on GitHubhttps://github.com//issues/112#issuecomment-8341086.


This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.

@kwallis
Copy link
Member Author

kwallis commented Sep 6, 2012

If we want to do this, I don't think it can be a

Ken Wallis

Product Manager – BlackBerry WebWorks

Research In Motion

(905) 629-4746 x14369


From: Ken Wallis
Sent: Thursday, September 06, 2012 1:59 PM
To: blackberry/BB10-WebWorks-Framework; blackberry/BB10-WebWorks-Framework
Subject: RE: [BB10-WebWorks-Framework] As a WebWorks developer, I can enable string pattern matching in my content (#112)

I completely agree. At the same time, it seems backwards to be adding a feature to the config, with a param, only to DISABLE a feature. Right now I am leaning to that being just too weird...

Ken Wallis

Product Manager – BlackBerry WebWorks

Research In Motion

(905) 629-4746 x14369


From: Nukul Bhasin [[email protected]]
Sent: Thursday, September 06, 2012 1:50 PM
To: blackberry/BB10-WebWorks-Framework
Cc: Ken Wallis
Subject: Re: [BB10-WebWorks-Framework] As a WebWorks developer, I can enable string pattern matching in my content (#112)

just think its better usability as more apps will want it enabled


Reply to this email directly or view it on GitHubhttps://github.com//issues/112#issuecomment-8341086.


This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.

@nukulb
Copy link
Contributor

nukulb commented Sep 6, 2012

I would disagree with that

From: Ken Wallis <[email protected]mailto:[email protected]>
Reply-To: blackberry/BB10-WebWorks-Framework <[email protected]mailto:[email protected]>
Date: Thu, 6 Sep 2012 11:01:34 -0700
To: blackberry/BB10-WebWorks-Framework <[email protected]mailto:[email protected]>
Cc: Nukul Bhasin <[email protected]mailto:[email protected]>
Subject: Re: [BB10-WebWorks-Framework] As a WebWorks developer, I can enable string pattern matching in my content (#112)

If we want to do this, I don't think it can be a

Ken Wallis

Product Manager – BlackBerry WebWorks

Research In Motion

(905) 629-4746 x14369


From: Ken Wallis
Sent: Thursday, September 06, 2012 1:59 PM
To: blackberry/BB10-WebWorks-Framework; blackberry/BB10-WebWorks-Framework
Subject: RE: [BB10-WebWorks-Framework] As a WebWorks developer, I can enable string pattern matching in my content (#112)

I completely agree. At the same time, it seems backwards to be adding a feature to the config, with a param, only to DISABLE a feature. Right now I am leaning to that being just too weird...

Ken Wallis

Product Manager – BlackBerry WebWorks

Research In Motion

(905) 629-4746 x14369


From: Nukul Bhasin [[email protected]:[email protected]]
Sent: Thursday, September 06, 2012 1:50 PM
To: blackberry/BB10-WebWorks-Framework
Cc: Ken Wallis
Subject: Re: [BB10-WebWorks-Framework] As a WebWorks developer, I can enable string pattern matching in my content (#112)

just think its better usability as more apps will want it enabled


Reply to this email directly or view it on GitHubhttps://github.com//issues/112#issuecomment-8341086.


This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.


Reply to this email directly or view it on GitHubhttps://github.com//issues/112#issuecomment-8341461.


This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.

@kwallis
Copy link
Member Author

kwallis commented Sep 6, 2012

Right, but that is all you would ever see in config for string pattern, is to disable it. It is just backwards...

Ken Wallis

Product Manager – BlackBerry WebWorks

Research In Motion

(905) 629-4746 x14369


From: Nukul Bhasin [[email protected]]
Sent: Thursday, September 06, 2012 2:04 PM
To: blackberry/BB10-WebWorks-Framework
Cc: Ken Wallis
Subject: Re: [BB10-WebWorks-Framework] As a WebWorks developer, I can enable string pattern matching in my content (#112)

I would disagree with that

From: Ken Wallis <[email protected]mailto:[email protected]>
Reply-To: blackberry/BB10-WebWorks-Framework <[email protected]mailto:[email protected]>
Date: Thu, 6 Sep 2012 11:01:34 -0700
To: blackberry/BB10-WebWorks-Framework <[email protected]mailto:[email protected]>
Cc: Nukul Bhasin <[email protected]mailto:[email protected]>
Subject: Re: [BB10-WebWorks-Framework] As a WebWorks developer, I can enable string pattern matching in my content (#112)

If we want to do this, I don't think it can be a

Ken Wallis

Product Manager – BlackBerry WebWorks

Research In Motion

(905) 629-4746 x14369


From: Ken Wallis
Sent: Thursday, September 06, 2012 1:59 PM
To: blackberry/BB10-WebWorks-Framework; blackberry/BB10-WebWorks-Framework
Subject: RE: [BB10-WebWorks-Framework] As a WebWorks developer, I can enable string pattern matching in my content (#112)

I completely agree. At the same time, it seems backwards to be adding a feature to the config, with a param, only to DISABLE a feature. Right now I am leaning to that being just too weird...

Ken Wallis

Product Manager – BlackBerry WebWorks

Research In Motion

(905) 629-4746 x14369


From: Nukul Bhasin [[email protected]:[email protected]]
Sent: Thursday, September 06, 2012 1:50 PM
To: blackberry/BB10-WebWorks-Framework
Cc: Ken Wallis
Subject: Re: [BB10-WebWorks-Framework] As a WebWorks developer, I can enable string pattern matching in my content (#112)

just think its better usability as more apps will want it enabled


Reply to this email directly or view it on GitHubhttps://github.com//issues/112#issuecomment-8341086.


This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.


Reply to this email directly or view it on GitHubhttps://github.com//issues/112#issuecomment-8341461.


This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.


Reply to this email directly or view it on GitHubhttps://github.com//issues/112#issuecomment-8341575.


This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.

@nukulb
Copy link
Contributor

nukulb commented Sep 6, 2012

fine by me, just imagine that every app needs to add it.

@cdelcol
Copy link
Contributor

cdelcol commented Sep 28, 2012

@jrogers is making updates to this as of today. Make sure to touch base with him before implementation.

@jrogers
Copy link

jrogers commented Sep 28, 2012

We already have an existing "InvokeRequest" event that gets emitted by the WebView that will probably be used for this. Basically you will need to listen for that event and process it/pass it to invoke or whatever. It's the same mechanism for string patterns.

@nukulb
Copy link
Contributor

nukulb commented Sep 28, 2012

@jeffheifetz should already know
~Nukul

From: Jeff Rogers [mailto:[email protected]]
Sent: Friday, September 28, 2012 02:57 PM
To: blackberry/BB10-WebWorks-Framework [email protected]
Cc: Nukul Bhasin
Subject: Re: [BB10-WebWorks-Framework] As a WebWorks developer, I can enable string pattern matching in my content (#112)

We already have an existing "InvokeRequest" event that gets emitted by the WebView that will probably be used for this. Basically you will need to listen for that event and process it/pass it to invoke or whatever. It's the same mechanism for string patterns.


Reply to this email directly or view it on GitHubhttps://github.com//issues/112#issuecomment-8996559.


This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.

@jrogers
Copy link

jrogers commented Sep 29, 2012

I should also add that this is a change. Before the browser was processing this in it's LocationChanging event handler. This is what I'm changing as the LocationChanging is a hack.

@jeffheifetz
Copy link
Contributor

We had a working implementation using the InvokeRequest event, it was waiting on a bug-fix in the pattern matching itself.

@jeffheifetz
Copy link
Contributor

Waiting on internal PR 173340.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants