Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Is a multi selector in fact a Selector? #26

Open
iherman opened this issue Nov 2, 2017 · 2 comments
Open

Is a multi selector in fact a Selector? #26

iherman opened this issue Nov 2, 2017 · 2 comments

Comments

@iherman
Copy link
Member

iherman commented Nov 2, 2017

(Taken from @tcole3’s comment)

@iherman
Copy link
Member Author

iherman commented Nov 2, 2017

Elsewhere @azaroth42 has said, "I think multiple selections should be at the SpecificResource [locator] level, the same way as we can have multiple targets in an Annotation." I would suggest that this is dependent on the answer to [issue #24]. If ERS is a valid Selector, then the multi Selector seems okay. But if ERS is not a valid type of Selector, then multi Selector is also not a Selector, and we would need to fall back on the multi-locator approach previously proposed.

(From @tcole3’s comment)

@iherman
Copy link
Member Author

iherman commented Nov 2, 2017

I am not sure the relation with #24 is so clear cut. Of course, if the ERS is not a valid Selector, then we have a problem with MultiSelector, too, which relies on it. However, even if ERS is a valid Selector, one could think of dumping a MultiSelector in favor of multiple locators.

That being said, it becomes also a convenience question. The Locator's usage/implementation may be simpler if everything is one locator; otherwise we are forced to define higher level notions on how locators are used (this is not a problem for WA, but we would have to do something extra here, if only by taking over more from WA). Ie, I prefer keeping the local structure, but, at the end of the day, end users as well as implementers may have to decide...

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

No branches or pull requests

2 participants