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 size and slight adjustments #17

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

NicolasAntoine
Copy link
Contributor

Modified the coping strategy text by removing 'suggested coping strategy' and enhanced the graph for better clarity regarding its purpose. Added the possibility to adjust the text size as well.

Copy link
Owner

@Vikranth3140 Vikranth3140 left a comment

Choose a reason for hiding this comment

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

Hey @NicolasAntoine,

Thank you for the PR, please mention which issue is this PR aimed at, and if no such issue exists, then please create a new issue and mention it.

@NicolasAntoine
Copy link
Contributor Author

Hey I don't really know how to mention an issue but my PR helps to resolve the issue #12. Let me know what you think of it ;)

@Vikranth3140
Copy link
Owner

Oh okay, please mention what accessibility features you have added and also send a screenshot of the updated application so that i can see the updated accessibility features.

@NicolasAntoine
Copy link
Contributor Author

image
I added a slide bar to adjust text size

@Vikranth3140
Copy link
Owner

I don't think this PR enhances and increases accessibility features, but is a code formatting PR.
So, just make an issue for the same and please mention what exactly has been changed.

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.

2 participants