Hi 47Deg developers!

In this PR I've changed your text validation policy in ClipboardServicesComponentImpl to fix the issue #7.

Take a look to the new conditions I've added. Now, if the text to translate is a number or a URL the text is not consider as valid. I've added some tests to ClipboardServiceComponentSpec.

I've used a regular expression to check if the text to translate is a URL or not and the forall functional combinator to check if the text is a number.

To write this tests I've used clipboardServices.isValidCall shouldEqual false but I'm not sure if this is the correct way to assert this expected value :S This is my first PR to a Scala project :)

Maybe all the code related to the text validation should be extracted to another class. But I think for this PR is not needed.

This PR adds the configuration file for CircleCI

Build CircleCI: https://circleci.com/gh/47deg/translate-bubble-android/6

Please @raulraja could you review? Thanks

This PR allows to send text to the translate service through the share option. In that way you can activate the bubble with the share option.

@javipacheco please could you review?

Thanks

This PR make a refactor of all UI actions.

• Methods returns Ui[_] and the runUi is done in the Activity or Fragment
• Created Composer trait with UI Actions and Views
• Sealed trait for types (Bubble and Notification)

@fedefernandez @franciscodr can you please review? Thanks!

Scala is the best, Java is dead

I have moved this code to Scala and I have enjoyed a lot :-)

• In Java I have used "Dagger" library for Dependency Injection and now I use Cake Pattern and Service[Req, Res]. The Cake Pattern components are in "modules" package
• In Java I have used "android-priority-jobqueue" library for async tasks and now I use Futures on Scala

I have some problems currently:

• I'm using implementations in my traits. For example, I don't know how I should use TranslateServicesComponent instead of TranslateServicesComponentImpl
• We should create a new module for "MacroidExtras" for we can use it in others projects. I have talked with @juanpedromoreno about that

@raulraja can you review please? Thanks!

@juanpedromoreno @FPerezP and @franciscodr Enjoy it!

To avoid useless requests and to not show the bubble when it's not needed, we should identify numbers, URLs and any other string that we should avoid to translate.

As an idea, we could have a regex list that allow us to define patterns that should not be translated.

This PR covers:

• Fixed some bugs
• Added Google Analytics
• I've added a new sections in README for how compile the project

@raulraja can you please review? It`s possible that I have forgotten something in compile section. Can you review if I forgot some important thing?

This PR upgrades the macroid and macroid-extra version and replaces the layouts in ActionsView and DisableView for macroid-ui using the new ServiceContextWrapper in the new version of macroid library

@fedefernandez can you please review? thanks!

In cell phones where the Clipboard Listener is called several times (like Nexus 6), the bubble doesn't appear sometimes

@franciscodr can you please review? thanks!

This PR solves:

• Wrong text shown for languages in translation window
• Unescaped encoded characters like &39; in translated text

@javipacheco, @raulraja Could you review this code, please? Thanks!

This is a Release Candidate! Yeah!!!

Israel have added new icon and new resources. I have fixed some bugs

@raulraja can you please review the code?

@justinelsberry can you please review 2 new string?

@jorgegalindocruces can you please review Spanish resources?

Thanks!

Thanks for the useful app! However, the default translation API was blocked in my country.

I hope this app can support more translation API providers, such as:

• Yandex(it seems free)
• Bing(It is free in 2M data)

Thanks.

Dear Team,

Thanks a lot for amazing useful app! However, the following improvements will make it much easier:

1. For translation "window" (which contains translation results): 1.1 Close the "window" by tap on any point on it (not on X button only). Motivation: it minimizes time to close cause doesn't need to aim into X button. Moreover, there is no functionality within "window" so it safe to tap inside 1.2 Close the "window" by tap anywhere on screen Motivation: minimize time to translate and continue reading ASAP.
2. For translation flow: 2.1 Show translation as soon as text copied to clipboard Motivation: In use case, if enable option or start the app before reading, a user will have ability to translate in less actions - just select and copy! Together with the mentioned options it makes translation almost unnoticeable during reading. Remark: It should be optional (disabled by default to avoid annoying for new users).