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

Added text alignment feature to editor and improved some styles for user-friendly interface #1782

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rezaei92
Copy link

According to my comment in PR #1752, the changes made are as follows:

We've used SVGs for the 'text alignment' newly added feature because Unicode characters doesn't exist for that and use SVGs for ordered, and unordered lists to provide a clearer and more user-friendly interface.

I fixed the SVG color for dark mode too.

The modifications in SquireUI.less were designed to streamline the editor's layout and enhance its overall aesthetics. Specifically, we adopted a flexbox-based approach for the toolbar buttons, replacing the previous scroll-based layout. Additionally, we ensured correct displaying of SVG icons and the button order for right-to-left (RTL) languages.

The adjustments made to NextcloudV25+/styles.css were aimed at preserving a consistent and user-friendly appearance across different themes by overriding certain default styles.

These changes will make to improve and more completely the editor and, I'll fix it If you have any ideas or suggestions for accepting this PR.

Thank you

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.

1 participant