-
Notifications
You must be signed in to change notification settings - Fork 67
add bitly command #48
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
base: master
Are you sure you want to change the base?
Conversation
sharmasha2nk
commented
Apr 23, 2016
- Support to add validations in Search Widget
- Using isResultList to change height of result container
- Update limit to 6 from 5
- Bitly api support
- Added valid-url dependency to validate URL
- Support to add validations in Search Widget - Using isResultList to change height of result container - Update limit to 6 from 5 - Bitly api support - Added valid-url dependency to validate URL
Thank you so much for this PR — I love the idea of adding bit.ly support to Command! Super excited to get this merged in. My primary concern is that I don't think What do you think about that? Would you be up for refactoring along those lines? Again — really excited to get this merged in! |
Glad you liked the idea. 😀 I was also thinking about adding another component with button. But I later go ahead adding validations in search which will hold making any network call until a valid link is entered. I think usually if people are using bitly to shorten link they will be copying the link instead of typing. And adding button after that will just increase the clicks. I know there is one issue right now that if someone start typing the link to shorten, it will make the shorten link once it found a valid URL even though he didn't finished typing URL 😰. I'll fix that issue in next commit. May be we can have something similar like Bitly site is having. If link is pasted will shorten it then and there otherwise user hits enter or button to shorten it. Let me know your views. Thanks. P.S I have also posted an article about your extension medium/sharmasha2nk/command |
I like the solution of automatically shortening on |
maybe |
Hi @jessepollak how about this UX: |
@sharmasha2nk yes, that looks awesome - is that in the current PR? |
@jessepollak Not yet in PR as I'm planning to add options page also so user can configure his own bitly account. Would be great if you could help to solve #51 |