Skip to content

MarkUnknown checkbox visible with TranslateDoc - Fix #146 #160

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.

Sign up for GitHub

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

Closed
wants to merge 10 commits into from
Closed

MarkUnknown checkbox visible with TranslateDoc - Fix #146 #160

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
caeff03
Mark Unknown with translate Doc
share-with-me Jul 1, 2017
f926fb3
Change ID to markUnknownDoc
share-with-me Jul 1, 2017
ded1c24
Add a class to hide and show elements
share-with-me Jul 6, 2017
1f2d73b
Change ID
share-with-me Jul 6, 2017
b8ae5b9
Use siblings menthod
share-with-me Jul 11, 2017
963ae01
Add functionality when markunknown is changed on docTranslation inter…
share-with-me Jul 16, 2017
9187c4e
Update index.html.in
sushain97 Jul 29, 2017
7a79a83
Use *:not selectors
share-with-me Jul 31, 2017
32f30cb
Merge branch 'master' into gsoc-mono-markunknown-translateDoc
share-with-me Jul 31, 2017
ec11119
Remove trailing spaces
share-with-me Jul 31, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add functionality when markunknown is changed on docTranslation inter… 
…face
  • Loading branch information
@share-with-me
share-with-me committed Jul 16, 2017
commit 963ae0176b1a0b838104845e7e170c307ac65fad
3 changes: 3 additions & 0 deletions assets/js/translator.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -147,6 +147,9 @@ if(modeEnabled('translation')) {
if($('div#translateText').is(':visible')) {
translateText();
}
else if($('div#docTranslation').is(':visible')) {
translateDoc();
}
Copy link
Contributor Author

@share-with-me share-with-me Jul 31, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Um, I am not sure why this is appearing in the diff here :/ . I simply did a git rebase with master. Do you suggest making a separate PR? (The below snippet has the changes that you asked to do @sushain97 )!

Copy link
Member

@sushain97 sushain97 Aug 2, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm, maybe you're rebasing with your master instead of origin/master? I think these changes are separate so we don't want to merge them.

Copy link
Contributor Author

@share-with-me share-with-me Aug 2, 2017 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Um, I did a git fetch origin, git rebase origin/master. I am not sure why it still exists. Is it okay if I create a new PR?

persistChoices('translator');
});

Expand Down