Skip to content

Comments

feat: accessability section#162

Closed
cWillow wants to merge 1 commit intomasterfrom
feat/accessability-section
Closed

feat: accessability section#162
cWillow wants to merge 1 commit intomasterfrom
feat/accessability-section

Conversation

@cWillow
Copy link
Contributor

@cWillow cWillow commented Sep 7, 2021

No description provided.

@cWillow cWillow requested a review from vpanjganj September 7, 2021 14:53
@cWillow cWillow force-pushed the feat/accessability-section branch 4 times, most recently from faf10f0 to 51c9715 Compare September 7, 2021 15:25
@cWillow cWillow marked this pull request as draft September 7, 2021 15:31
@cWillow cWillow force-pushed the feat/accessability-section branch 2 times, most recently from 5cd19fa to ed2a0ed Compare September 9, 2021 11:22
@cWillow cWillow force-pushed the feat/accessability-section branch from ed2a0ed to 562c14b Compare September 9, 2021 13:07
@vpanjganj
Copy link
Collaborator

vpanjganj commented Sep 20, 2021

  1. We should find a bigger icon.
  2. Try break down your points into subsections which will lie under 10. Making 10.1, 10.2, etc
  3. There is a line break before why
  4. Try to avoid nested Whys.

@anya91m
Copy link
Contributor

anya91m commented Sep 21, 2021

Subsections and line breaks added. Have also tried to remove nesting for the reasons.

@cWillow cWillow closed this Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vpanjganj vpanjganj Awaiting requested review from vpanjganj

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cWillow @vpanjganj @anya91m