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

Release 0.5.1 bookmarklet change Note Default text to placeholder #17

Closed
nitinthewiz opened this issue Feb 28, 2014 · 1 comment
Closed

Comments

@nitinthewiz
Copy link

Hi,

I was able to install 0.5.1 on my shared host and have been using it.

One issue - In the Bookmarklet, there's a section to Add a Note. When you click on that button (Why is it a button and not a text field instead?), you get a text field to enter your note. The text inside the text field says "Type note text here...". This text should be a placeholder and not actual text, as this adds an extra step of having to select that text and remove it.

I tried to look into the app to find the place where it's located (to make the changes myself) but only found the "Add a Note" button and not the text field code. Else I would love to start contributing code here, instead of just filing issues.

@timwco
Copy link
Contributor

timwco commented Feb 28, 2014

@nitinthewiz we are going to clean up that interaction a bit still. We are also not using a textarea or input field but rather a content-editable container so placeholder text isn't supported by default. We will make it much smoother though. We are still focused on some of the core features right now but will round back up on the front end experience shortly.

Also, all of that info can be found inside of the /views folder. Some of the content though is generated by JS and we use some JS templating as well so I do understand your confusion. Once we finish some of the bigger pieces, we will write up some solid documentation to better explain how we are accomplishing things.

As always, thanks for your input!

@timwco timwco closed this as completed Feb 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants