> I don't know why someone would want to override unselectable styles, but the > first idea that popped into my head was a different background color so folks > knew not to even bother to select text in the unselectable controls. Another > possibility is that there are other styles that may be needed someday that > control how screen readers deal with unselectable controls.
Ah. You mean add additional styles. There’s no reason they couldn’t add an additional .unselectable selector with styling elsewhere in the CSS. I could go either way on this. If folks feel strongly it should be a “proper” selector, I’ll do that. Thanks, Harbs > On Jun 4, 2018, at 7:07 PM, Alex Harui <aha...@adobe.com.INVALID> wrote: > > Feel free to change the name of the bead so that the Type Selector name looks > "better". I think we agreed to drop the "Bead" portion of names going > forward. > > If you are absolutely sure nobody will ever need to override what is in that > selector, then I'm fine with injecting it with code and using the new utility > function you created, but otherwise, I would hope we try to write the > framework without injecting styles with code. Having overrideable selectors > should be the recommended practice for a framework. We want everything to be > override-able. > > I don't know why someone would want to override unselectable styles, but the > first idea that popped into my head was a different background color so folks > knew not to even bother to select text in the unselectable controls. Another > possibility is that there are other styles that may be needed someday that > control how screen readers deal with unselectable controls. > > My 2 cents, > -Alex > > On 6/4/18, 8:48 AM, "Harbs" <harbs.li...@gmail.com > <mailto:harbs.li...@gmail.com>> wrote: > > That would probably work. I’m not totally happy with that selector name. > > I can switch the implementation if you think that’s better. I’m not sure > why a user would override an “unelectable” style though. > > Harbs > >> On Jun 4, 2018, at 6:45 PM, Alex Harui <aha...@adobe.com.INVALID> wrote: >> >> Hi Harbs, >> >> My suggestion for adding the selector is to make the selector look like a >> Type/Element Selector with the name "UnselectableElementBead". If you do >> that, then wouldn't that selector be dropped out if nobody is using >> UnselectableElementBead? >> >> I would hope we could find a solution that doesn't result in code injecting >> styles as that sort of screws up users overriding those styles. >> >> My 2 cents, >> -Alex >> >> On 6/4/18, 1:14 AM, "Harbs" <harbs.li...@gmail.com >> <mailto:harbs.li...@gmail.com> <mailto:harbs.li...@gmail.com >> <mailto:harbs.li...@gmail.com>>> wrote: >> >> It does add the style to the class list. >> >> The only question is how to add the selector to the CSS. >> >> Thanks, >> Harbs >> >>> On Jun 4, 2018, at 10:24 AM, Carlos Rovira <carlosrov...@apache.org >>> <mailto:carlosrov...@apache.org>> wrote: >>> >>> Hi, >>> >>> I think as Alex that better way for this is the bead add the css style to >>> classList. >>> >>> thanks >>> >>> 2018-06-04 6:33 GMT+02:00 Alex Harui <aha...@adobe.com.invalid >>> <mailto:aha...@adobe.com.invalid>>: >>> >>>> IMO, the better fix would be to just rename the selector in defaults.css >>>> to be UnselectableElementBead and have UnselectableElementBead add >>>> "UnselectableElementBead" to the classList. >>>> >>>> Or am I not understanding what this does? >>>> >>>> -Alex >>>> >>>> >>>> On 6/2/18, 1:59 PM, "Harbs" <harbs.li...@gmail.com >>>> <mailto:harbs.li...@gmail.com>> wrote: >>>> >>>> What I really wanted to do here was create some self invoking code >>>> which was run when the UnselectableElementBead class was loaded. >>>> >>>> Is there any way of doing that without writing a static method and >>>> invoking that? >>>> >>>> Thanks, >>>> Harbs >>>> >>>> >>>>> On Jun 2, 2018, at 11:39 PM, ha...@apache.org <mailto:ha...@apache.org> >>>>> wrote: >>>>> >>>>> This is an automated email from the ASF dual-hosted git repository. >>>>> >>>>> harbs pushed a commit to branch develop >>>>> in repository https://na01.safelinks.protection.outlook.com/?url= >>>>> <https://na01.safelinks.protection.outlook.com/?url=> >>>> https%3A%2F%2Fgitbox.apache.org >>>> <http://2fgitbox.apache.org/>%2Frepos%2Fasf%2Froyale- >>>> asjs.git&data=02%7C01%7Caharui%40adobe.com >>>> <http://40adobe.com/>%7C9369538a50114755520308d5c8cb >>>> b09c%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0% >>>> 7C636635699536531937&sdata=iPymBmOQfeD1312Szlsc6Fwc4E5GU2 >>>> Kj3OwfvmWh%2Bzk%3D&reserved=0 >>>>> >>>>> >>>>> The following commit(s) were added to refs/heads/develop by this >>>> push: >>>>> new b157ecf Removed unselectable selector >>>>> b157ecf is described below >>>>> >>>>> commit b157ecff5ea11ff2035e105f266b0925c1970005 >>>>> Author: Harbs <ha...@in-tools.com <mailto:ha...@in-tools.com>> >>>>> AuthorDate: Sat Jun 2 23:39:28 2018 +0300 >>>>> >>>>> Removed unselectable selector >>>>> --- >>>>> .../projects/Basic/src/main/resources/defaults.css | 9 --------- >>>>> .../royale/html/beads/UnselectableElementBead.as | 22 >>>> ++++++++++++++++++++++ >>>>> 2 files changed, 22 insertions(+), 9 deletions(-) >>>>> >>>>> diff --git a/frameworks/projects/Basic/src/main/resources/defaults.css >>>> b/frameworks/projects/Basic/src/main/resources/defaults.css >>>>> index ea02120..23a324d 100644 >>>>> --- a/frameworks/projects/Basic/src/main/resources/defaults.css >>>>> +++ b/frameworks/projects/Basic/src/main/resources/defaults.css >>>>> @@ -26,15 +26,6 @@ >>>>> box-sizing: border-box; >>>>> } >>>>> >>>>> -.unselectable >>>>> -{ >>>>> - -moz-user-select: -moz-none; >>>>> - -khtml-user-select: none; >>>>> - -webkit-user-select: none; >>>>> - -o-user-select: none; >>>>> - user-select: none; >>>>> -} >>>>> - >>>>> Alert >>>>> { >>>>> IBeadLayout: ClassReference("org.apache.royale.html.beads.layouts. >>>> VerticalFlexLayout"); >>>>> diff --git a/frameworks/projects/Basic/src/main/royale/org/apache/ >>>> royale/html/beads/UnselectableElementBead.as b/frameworks/projects/Basic/ >>>> src/main/royale/org/apache/royale/html/beads/UnselectableElementBead.as >>>>> index 6697472..c80c907 100644 >>>>> --- a/frameworks/projects/Basic/src/main/royale/org/apache/ >>>> royale/html/beads/UnselectableElementBead.as >>>>> +++ b/frameworks/projects/Basic/src/main/royale/org/apache/ >>>> royale/html/beads/UnselectableElementBead.as >>>>> @@ -34,6 +34,27 @@ package org.apache.royale.html.beads >>>>> public class UnselectableElementBead implements IBead >>>>> { >>>>> /** >>>>> + * @royaleignorecoercion HTMLStyleElement >>>>> + */ >>>>> + private static function insertRule():void >>>>> + { >>>>> + // only do this once... >>>>> + if(ruleInserted) >>>>> + return; >>>>> + ruleInserted = true; >>>>> + // Inject a new css selector >>>>> + COMPILE::JS >>>>> + { >>>>> + var style:HTMLStyleElement = >>>> document.createElement('style') as HTMLStyleElement; >>>>> + style.type = 'text/css'; >>>>> + style.innerHTML = '.unselectable >>>> {-moz-user-select: none;-webkit-user-select: none;-ms-user-select: >>>> none;user-select: none;}'; >>>>> + document.getElementsByTagName( >>>> 'head')[0].appendChild(style); >>>>> + } >>>>> + >>>>> + } >>>>> + private static var ruleInserted:Boolean; >>>>> + >>>>> + /** >>>>> * constructor. >>>>> * >>>>> * @langversion 3.0 >>>>> @@ -57,6 +78,7 @@ package org.apache.royale.html.beads >>>>> */ >>>>> public function set strand(value:IStrand):void >>>>> { >>>>> + insertRule(); >>>>> _strand = value; >>>>> >>>>> COMPILE::JS >>>>> >>>>> -- >>>>> To stop receiving notification emails like this one, please contact >>>>> ha...@apache.org <mailto:ha...@apache.org>. >>>> >>>> >>>> >>>> >>> >>> >>> -- >>> Carlos Rovira >>> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fabout.me%2Fcarlosrovira&data=02%7C01%7Caharui%40adobe.com%7C0bd6353a12104f5326d708d5c9f332c7%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636636968773528501&sdata=v8vk8mHKjoAN3D0lexy3vhXcOWVeYYc1HDouygyKoE8%3D&reserved=0 >>> >>> <https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fabout.me%2Fcarlosrovira&data=02%7C01%7Caharui%40adobe.com%7C0bd6353a12104f5326d708d5c9f332c7%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636636968773528501&sdata=v8vk8mHKjoAN3D0lexy3vhXcOWVeYYc1HDouygyKoE8%3D&reserved=0><https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fabout.me%2Fcarlosrovira&data=02%7C01%7Caharui%40adobe.com%7C0bd6353a12104f5326d708d5c9f332c7%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636636968773528501&sdata=v8vk8mHKjoAN3D0lexy3vhXcOWVeYYc1HDouygyKoE8%3D&reserved=0 >>> >>> <https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fabout.me%2Fcarlosrovira&data=02%7C01%7Caharui%40adobe.com%7C0bd6353a12104f5326d708d5c9f332c7%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636636968773528501&sdata=v8vk8mHKjoAN3D0lexy3vhXcOWVeYYc1HDouygyKoE8%3D&reserved=0>>