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

User-friendliness: Add labels and change "more info" button for team contribution questions #12153

Merged
merged 2 commits into from
Mar 3, 2023

Conversation

weiquu
Copy link
Contributor

@weiquu weiquu commented Mar 2, 2023

Part of #12081
Sub-issue: Labels and "more info" button for team contribution question

Outline of Solution

Add labels similar to previous question types. Also changed the position of the "more info" button as I think that it makes more sense to come under question instructions. In addition, since tabindex is a global attribute, to allow for the button to come first while in its current position, there would be a need to add/change the tabindex of many elements on the page. Moving the "more info" button solves this problem.

I also altered the CSS a little to take into account the move. See below for a comparison.

Previously (v4):
v4

After the change:
current

Copy link
Contributor

@zhaojj2209 zhaojj2209 left a comment

Choose a reason for hiding this comment

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

LGTM!

@zhaojj2209 zhaojj2209 added the s.ToMerge The PR is approved by all reviewers including final reviewer; ready for merging label Mar 3, 2023
@zhaojj2209 zhaojj2209 merged commit a863cd1 into TEAMMATES:user-friendliness Mar 3, 2023
zhaojj2209 pushed a commit that referenced this pull request Mar 23, 2023
…contribution questions (#12153)

* Add labels and change button

* Update tests
zhaojj2209 pushed a commit that referenced this pull request Mar 23, 2023
…for team contribution questions (#12153)

* Add labels and change button

* Update tests
zhaojj2209 pushed a commit that referenced this pull request Mar 25, 2023
…for team contribution questions (#12153)

* Add labels and change button

* Update tests
zhaojj2209 pushed a commit that referenced this pull request Mar 25, 2023
…for team contribution questions (#12153)

* Add labels and change button

* Update tests
@zhaojj2209 zhaojj2209 added the c.Feature User-facing feature; can be new feature or enhancement to existing feature label Apr 2, 2023
@zhaojj2209 zhaojj2209 self-assigned this Apr 2, 2023
@zhaojj2209 zhaojj2209 added this to the V8.26.0 milestone Apr 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c.Feature User-facing feature; can be new feature or enhancement to existing feature s.ToMerge The PR is approved by all reviewers including final reviewer; ready for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants