I've added ids to all those buttons(already merged in). This is copied from Damian's pull request because I don't want to run into merge conflict. I will change it once both pull request gets merged in. 

----- Original Message -----

> From: "David Mason" <notifications@github.com>
> To: "zanata/zanata-server" <zanata-server@noreply.github.com>
> Cc: "Patrick Huang" <pahuang@redhat.com>
> Sent: Friday, February 7, 2014 4:04:43 PM
> Subject: Re: [zanata-server] Gettext plural and properties functional test
> (#353)

> In functional-test/src/main/java/org/zanata/page/webtrans/EditorPage.java:
> > +        });
> > +        we.click();
> > +        we.clear();
> > +        we.sendKeys(text);
> > +    }
> > +
> > +    /**
> > +     * Press the Approve button for the currently selected translation row
> > +     * @return new Editor page object
> > +     */
> > +    public EditorPage approveSelectedTranslation() {
> > +        WebElement approve = waitForTenSec().until(new Function<WebDriver,
> > WebElement>() {
> > +            @Override
> > +            public WebElement apply(WebDriver input) {
> > +                return input.findElement(By.className("selected"))
> > +                        .findElement(By.className("icon-install"));

> Ideally we would have another class on each of the buttons that is
> independent of the particular icon we are using, to make this code more
> robust and readable.

> —
> Reply to this email directly or view it on GitHub .