-
Notifications
You must be signed in to change notification settings - Fork 66
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
Proposal: Enhance navigation in the root mouse
package
#362
Labels
RFC
Proposals to discuss
Comments
I wonder if we can gather some data from mouse usage in OS codebases to inform this, ie understand how much is |
We can do weak analytics via search on Github. And on the whole, I think we can save |
Ok, those search results reveal much more targeted use of mouse imports
than I'd expected. I tend to import `all` myself. Suggesting that some
emphasis on convenient groupings of mouse functionality would be
appreciated by users.
…On Tue, 30 Aug 2022, 4:58 pm Daniel Esik, ***@***.***> wrote:
We can do weak analytics via search on Github
<https://github.com/search?l=Scala&q=%22import+mouse%22&type=Code>. And
on the whole, I think we can save mouse.all, because it's kinda
super-import of all syntaxes.
—
Reply to this email directly, view it on GitHub
<#362 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAXJZHRCEWUZCSCXDKP6QTV3WWJTANCNFSM576MMYKQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
@benhutchison thanks for your thoughts! I'll return with draft PR after a while then. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There are a bunch of different syntaxes in the
mouse
- for boxed primitive-like data types (Int
,Long
,Boolean
, etc), for collections (Map
,Set
,List
, etc), for nested things (F[Either[A, B]]
,F[Option[A]]
, etc). And as you probably already guessed, we can combine some of those in groups -mouse.collections
,mouse.nested
, etc. It'd be a bit nicer from a design perspective, as well as from navigation. But this would require us to do a new major release since that change isn't binary compatible. What do you think about it?The text was updated successfully, but these errors were encountered: