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

fix(docs): fix documentation rtl for dialog #5924

Merged
merged 1 commit into from
Jul 10, 2021

Conversation

N1XUS
Copy link
Contributor

@N1XUS N1XUS commented Jul 9, 2021

Please provide a link to the associated issue.

Closes #5792

Please provide a brief summary of this pull request.

Added scoped DialogService to the example container component in order for DialogService to have the same RtlService as the component.

Please check whether the PR fulfills the following requirements

Documentation checklist:

@N1XUS N1XUS self-assigned this Jul 9, 2021
@N1XUS N1XUS added bug Something isn't working denoland-porto Denoland sprint 66 Porto - bugfixing documentation There is an issue with documentation labels Jul 9, 2021
@N1XUS N1XUS added this to the Sprint 66 - Porto milestone Jul 9, 2021
@netlify
Copy link

netlify bot commented Jul 9, 2021

✔️ Deploy Preview for fundamental-ngx ready!

🔨 Explore the source changes: 16f505a

🔍 Inspect the deploy log: https://app.netlify.com/sites/fundamental-ngx/deploys/60e8840344a1d70008e36ba2

😎 Browse the preview: https://deploy-preview-5924--fundamental-ngx.netlify.app

@droshev droshev force-pushed the fix/5792-fix-documentation-rtl branch from 12c500d to 16f505a Compare July 9, 2021 17:14
@droshev droshev merged commit ef9b248 into main Jul 10, 2021
@droshev droshev deleted the fix/5792-fix-documentation-rtl branch July 10, 2021 00:57
maxym-valor pushed a commit to maxym-valor/fundamental-ngx that referenced this pull request Jul 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working denoland-porto Denoland sprint 66 Porto - bugfixing documentation There is an issue with documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RTL for overlays, popovers, dialogs
3 participants