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

tr-888/right-to-left-review-panel #398

Closed
wants to merge 1 commit into from

Conversation

marpesia
Copy link
Contributor

@marpesia marpesia commented Jun 7, 2021

Related to TR-888

To test it:

  • Run as guest with according locale in generis.conf.php: define('DEFAULT_ANONYMOUS_INTERFACE_LANG','ar-arb'); at line 47

Copy link
Contributor

@btamas btamas left a comment

Choose a reason for hiding this comment

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

  • The name of the branch is not conventional. It should start with fix, feature etc, not with ticket number.
  • Description should contain what did you modified. "Implement RTL mode of blablabla"
  • I have problem with design as well. If you check my screenshot, some part is misaligned:
    • the up/down arrow that opens a section
    • the stat numbers (1/17 etc)
    • stat number near Overview

Screen Shot 2021-06-08 at 08 00 46

@marpesia marpesia marked this pull request as draft June 8, 2021 06:09
@marpesia marpesia closed this Jun 8, 2021
@marpesia marpesia deleted the tr-888/right-to-left-review-panel branch June 8, 2021 06:21
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.

None yet

2 participants