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

Align left should specifically add ql-align-left class that sets alignment #4288

Open
enzedonline opened this issue Jul 4, 2024 · 0 comments · May be fixed by #4290
Open

Align left should specifically add ql-align-left class that sets alignment #4288

enzedonline opened this issue Jul 4, 2024 · 0 comments · May be fixed by #4290

Comments

@enzedonline
Copy link

When selected align left from the toolbar, it just removes any ql-align class.

'left' is missing from the whitelist, the align left button has no data-value attribute, so I guess this is sending a blot.format('align', false) (??)

This is based on an assumption that the default paragraph style is left aligned.

Many sites will use justify as the default alignment. The result in this case is that selecting left-aligned has no effect, the text remains justified.

Applying a left align style to a paragraph should do exactly that.

Solution:
Add 'left' to whitelist. Alignments should toggle if this is needed (ie applying center-align to a paragraph already center aligned should remove the align class).

@sidharthbh8 sidharthbh8 linked a pull request Jul 4, 2024 that will close this issue
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 a pull request may close this issue.

1 participant