On Thu, 2012-08-30 at 20:52 +0000, Manal M. Alhassoun wrote: > > Hello, > > Could you please review the attached patch.
Looks reasonable to me. Or maybe if (Application::GetSettings().GetLayoutRTL() != bLayoutRTL) pFilterBox->EnableMirroring(); is a little shorter with the same effect ? Either way, could you add yourself to http://wiki.documentfoundation.org/Development/Developers and send a mail like http://lists.freedesktop.org/archives/libreoffice/2012-July/035301.html to this list to state that your contributions are under our preferred MPL+/LGPLv3+ license combination, and then we can merge it in. C. _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice