Oh, also, I'm SUPER wary of actually running that bookmarklet because
of the fact it is injecting javascript directly into whatever page I
am viewing. This compromises all security if you ever put malicious
code in your app. Not only is there the issue of you being able to
know what sites I visit and bookmark, but you have the power to get my
passwords and credit card numbers if you wanted to and if I clicked on
your bookmarklet on a page that I enter such information.

So I don't know how many users are going to be willing to trust your
app. I don't think I will unless myself or another third party goes
through your code to be sure you aren't trying to get data from input
fields. It would be different if you were a big, well known company
providing this app. If google, yahoo, MS, or even much smaller
companies were offering this, then it could be trusted because no well
established company is going to publish malicious code without reaping
huge consequences.

my 2 cents.

On Jun 12, 7:17 pm, Sky <[email protected]> wrote:
> I quickly checked out the link tohttp://translatebookmarklet.appspot.com/
> I have not tried using the actual app yet, but I see several things
> that could be improved from the start.
>
> The bookmarklet code:
> javascript:(function(){
>         if(document.getElementById('GTranslatorscript')==null){
>                 gtb=document.createElement('SCRIPT');
>                 gtb.type='text/javascript';
>                 gtb.id='GTranslatorscript';
>                 gtb.src='http://translatebookmarklet.appspot.com/gtb.js';
>                 document.getElementsByTagName('head')[0].appendChild(gtb);
>         }else{
>                 window.alert('GTranslator is already loaded!');}
>         })();"
>
> You should NOT user the alert() to tell the user that it is already
> loaded. That's just annoying. Just don't do anything if it's already
> loaded.
>
> Your youtube video needs audio to explain what you are doing, and more
> zooming in to see what you are typing... I couldn't tell exactly what
> was going on.
>
> Your homepage needs to be really cleaned up and made more professional
> "Bookmarklet gonna save your language preferences" <- using the word
> "gonna" is terrible!
>
> Sorry, I don't have time right now to check out the actual product.
> Maybe later. Just remember, first impressions are incredibly
> important... including to get my interest to check it out and give you
> feedback. I'd be much more willing to look for bugs and problems if it
> seemed like you had a finished product made to look real professional.
>
> gl and sry for only negative feedback.... if the app works like it's
> supposed to then I bet it's awesome, so good job and keep up the hard
> work!
>
> cheers ^_^
>
> On Jun 12, 6:55 pm, KeremTiryaki <[email protected]> wrote:
>
>
>
> > Hello
> >   I have developed a translate bookmarklet with gwt.
> > Could you please try and send some feedback 
> > about?http://translatebookmarklet.appspot.com/
>
> > Its server side is app engine
> > I have usedhttp://code.google.com/p/google-api-translate-java/
> > JSONP for cross-domain communication.
> > It listens mouse selection and keyboard for rapid results. also saves
> > the user preferences, If user logged.

-- 
You received this message because you are subscribed to the Google Groups 
"Google Web Toolkit" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.

Reply via email to