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

The cursor changes to pointer when hovered over select element #923

Closed
wants to merge 3 commits into from
Closed

The cursor changes to pointer when hovered over select element #923

wants to merge 3 commits into from

Conversation

rohithGoudM
Copy link

Fixes #913

This is my first PR, so please let me know even if it is wrong.
I really want to contribute here (publiclab)
Any kind of reply would be greatly appreciated

Thanks!

@welcome
Copy link

welcome bot commented Mar 23, 2019

Thanks for opening this pull request!
There may be some errors, but don't worry! We're here to help! 👍🎉😄

@harshkhandeparkar
Copy link
Member

Ok you opened a pr. @publiclab/is-reviewers will review your pr.

Copy link
Member

@harshkhandeparkar harshkhandeparkar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you put the styles in examples/demo.css ?

@subhahu123
Copy link

@rohithGoudM can u provide a preview link
u can use github pages

@rohithGoudM
Copy link
Author

Can you put the styles in examples/demo.css ?

Yes,
But I also want to know what should I do to get into publiclab for gsoc2019.

@harshkhandeparkar
Copy link
Member

@rohithGoudM

Yes,
But I also want to know what should I do to get into publiclab for gsoc2019.

Nothing. Wait for this pr to be merged. You will get an invite to join publiclab and you can participate.

@harshkhandeparkar
Copy link
Member

You should know the codebase well. You can solve a few issues and you will be well acquainted with the codebase. 😊. I don't know much about gsoc and the proposals etc. You can ask @gauravano @jywarren @tech4GT or other @publiclab/is-reviewers

@publiclab publiclab deleted a comment from subhahu123 Mar 23, 2019
@subhahu123
Copy link

subhahu123 commented Mar 23, 2019

Can you put the styles in examples/demo.css ?

Yes,
But I also want to know what should I do to get into publiclab for gsoc2019.

as far as i know ( i m also applying ) u should continue contributing to publiclab and select ur project u want to work on and prepare a propoal for that
Also u should know the codebase well as said by @harshkhandeparkar

@publiclab publiclab deleted a comment from subhahu123 Mar 23, 2019
@rohithGoudM
Copy link
Author

Can you put the styles in examples/demo.css ?

Yes,
But I also want to know what should I do to get into publiclab for gsoc2019.

as far as i know ( i m also applying ) u should continue contributing to publiclab and select ur project u want to work on and prepare a propoal for that

I want to work on image-sequencer
I like the idea but how can i make any impression on publiclab's recruiting team

@subhahu123
Copy link

Can you put the styles in examples/demo.css ?

Yes,
But I also want to know what should I do to get into publiclab for gsoc2019.

as far as i know ( i m also applying ) u should continue contributing to publiclab and select ur project u want to work on and prepare a propoal for that

I want to work on image-sequencer
I like the idea but how can i make any impression on publiclab's recruiting team

ur contibution is only way to get an impression and then after ur proposal idea

@rohithGoudM
Copy link
Author

Can you put the styles in examples/demo.css ?

Done.

@rohithGoudM
Copy link
Author

Is my PR eligible for merging..?
Can I solve other issues that has first-timers-only issues..?
If not could you refer some issues here so I can try solving them..?

@subhahu123
Copy link

subhahu123 commented Mar 23, 2019

Is my PR eligible for merging..?
Can I solve other issues that has first-timers-only issues..?
If not could you refer some issues here so I can try solving them..?

#924
Try solve this one

@harshkhandeparkar
Copy link
Member

harshkhandeparkar commented Mar 23, 2019

You can solve any issue. I don't think there are any more First-Timers-Only issues left but there may be some which are marked as fto-candidate issues and also there are some with the label small. Also which field are you best in? Js/html/css? We can assign you issues based on that. If you are good with js, you can see check out some core library issues with the module label.

@rohithGoudM
Copy link
Author

Is my PR eligible for merging..?
Can I solve other issues that has first-timers-only issues..?
If not could you refer some issues here so I can try solving them..?

#924
solve this one

ON it.

@rohithGoudM
Copy link
Author

You can solve any issue. I don't think there are any more First-Timers-Only issues left but there may be some which are marked as fto-candidate issues and also there are some with the label small. Also which field are you best in? Js/html/css? We can assign you issues based on that. Of you are good with js, you can see check out some core library issues with the module label.

I have a little background in js through node js. Also happy to help with html and css.

@harshkhandeparkar
Copy link
Member

@subhahu123 is this ready to merge? Have you checked out the changes locally?

@subhahu123
Copy link

@subhahu123 is this ready to merge? Have you checked out the changes locally?

i have not checked this locally

@subhahu123
Copy link

@harshkhandeparkar
what are the changes in package.json
it was not required

@subhahu123
Copy link

i have just notice this

@subhahu123
Copy link

moreover he has to squash his commit also
@harshkhandeparkar

Copy link
Member

@harshkhandeparkar harshkhandeparkar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can see that the dist files have been changed directly. Please make the appropriate changes to the src files in examples/lib directory as well.

@rohithGoudM
Copy link
Author

@harshkhandeparkar
what are the changes in package.json
it was not required

those changes were in package-lock.json not in package.json,
I didn't know if those changes were useful.

@harshkhandeparkar
Copy link
Member

No need to squash commits. They are squashed while merging.

@subhahu123
Copy link

No need to squash commits. They are squashed while merging.

Ok

@rohithGoudM
Copy link
Author

I can see that the dist files have been changed directly. Please make the appropriate changes to the src files in examples/lib directory as well.

I have added the class to examples/lib files too

@subhahu123
Copy link

I can see that the dist files have been changed directly. Please make the appropriate changes to the src files in examples/lib directory as well.

I have added the class to examples/lib files too

where there is a change in package.json

@rohithGoudM
Copy link
Author

I have added the class to examples/lib files too

where there is a change in package.json

I didn't get you exactly.

@subhahu123
Copy link

I have added the class to examples/lib files too

where there is a change in package.json

I didn't get you exactly.

what changes u have done in package.json
i was not required u should remove it
@rohithGoudM

@aashna27
Copy link

"csrtoptr" can the class name be made a little more descriptive ??

@grvsachdeva
Copy link
Member

hey @rohithGoudM, you need any help finishing this one? Thanks!

@harshkhandeparkar
Copy link
Member

This has been fixed already.

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

Successfully merging this pull request may close these issues.

cursor should change into pointer
5 participants