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

Great job! #2

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Great job! #2

wants to merge 5 commits into from

Conversation

chytreg
Copy link

@chytreg chytreg commented Oct 17, 2010

Hi!
I made a lot of changes.
As for me the most useful is to select text in textmate hit CMD-SHIFT-I and put it in yml file.

You always should you ya2yaml instead of to_yml because has better support for UTF-8 characters!

Next I did refactor of yours code and added possibility to edit translations according to token key.

Select a token and hit CMD-SHIFT-E will try to find an associated translation and give you a possibility to change :)

I also added a simple command that open bundle config.yml i TextMate.
I left only shortcut for inserting and editing because I has already bind to something else.
MyGenego should works but I'm really not using it so as for me is not usefull.

I had to change a token key from t('.my_token') to t('views.admin.users.new.my_token') cause that shortcut doesn't work even it is a symbol.

Please look through the code and give me a feedback!

Review on Reviewable

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

Successfully merging this pull request may close these issues.

1 participant