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.

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

Jump to bottom

Ads display error #2227

Closed
ToufiqSuhail opened this issue Jun 6, 2018 · 3 comments
Closed

Ads display error #2227

ToufiqSuhail opened this issue Jun 6, 2018 · 3 comments
Assignees
@Sejiro-Hiko
Milestone
W14

Comments

@ToufiqSuhail
Copy link

Adsense problem with RTL & menu in the Arabic language

Ref: https://secure.helpscout.net/conversation/590799201/27070?folderId=2075531

Solution: Add Source code - .amp-ad-wrapper{direction:ltr;} in Custom CSS
@ahmedkaludi ahmedkaludi added this to the W14 milestone Jun 6, 2018
@Sejiro-Hiko
Copy link
Contributor

apparently, the error is only in SWIFT while using RTL

Sejiro-Hiko added a commit that referenced this issue Jun 8, 2018
@Sejiro-Hiko
#2227 Ad issue fixed
126dfe1
@MohammedKaludi
Copy link
Collaborator

the condition added will only work for AD 1, not for all ads. suggest you to remove the condition so it can work with all ads.

@MohammedKaludi MohammedKaludi mentioned this issue Jun 11, 2018
Closed
@MohammedKaludi
Copy link
Collaborator

Fixed in #2138

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Sejiro-Hiko Sejiro-Hiko

Labels
None yet
Projects
None yet
Milestone
W14
Development

No branches or pull requests
4 participants
@ahmedkaludi @MohammedKaludi @Sejiro-Hiko @ToufiqSuhail