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.

Sign up for GitHub

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

Jump to bottom

[PE-D][Tester E] Clicking on both lists highlights items in both lists #247

Closed
soc-se-bot opened this issue Mar 31, 2023 · 1 comment · Fixed by #264
Closed

[PE-D][Tester E] Clicking on both lists highlights items in both lists #247

soc-se-bot opened this issue Mar 31, 2023 · 1 comment · Fixed by #264
Assignees
@wz2k
Labels
Good to fix PE-D severity.VeryLow Purely cosmetic and does not affect usage type.Documentation Improvements or additions to documentation
Milestone
v1.4

Comments

@soc-se-bot
Copy link

Description

To navigate the list with up/down arrow keys, they first have to be clicked. Clicking both lists results in items of both lists being highlighted, which makes it unclear which list is currently selected.
image.png

Steps to reproduce

  1. Click on the elderly list
  2. Click on the volunteer list

Expected Behaviour

Selecting another list should unselect the previous list

Labels: severity.VeryLow type.FunctionalityBug
original: swxk19/ped#10
@wz2k wz2k self-assigned this Apr 1, 2023
@Zxun2 Zxun2 added Won't fix severity.VeryLow Purely cosmetic and does not affect usage PE-D labels Apr 2, 2023
@Zxun2
Copy link
Member

Zxun2 commented Apr 2, 2023

Solution

Document in UG. The last highlighted list will be used for navigation.

@Zxun2 Zxun2 added type.Documentation Improvements or additions to documentation Good to fix and removed Won't fix labels Apr 2, 2023
@wz2k wz2k mentioned this issue Apr 4, 2023
Merged
@wz2k wz2k added this to the v1.4 milestone Apr 4, 2023
@wz2k wz2k closed this as completed in #264 Apr 4, 2023
wz2k added a commit that referenced this issue Apr 4, 2023
@wz2k
Update ug with PED fixes (#264)
ba375dc 
# Changes
- Made find command more clear
- Removed old dg
- Update command summary

resolves #214, resolves #235, resolves #247, resolves #250
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wz2k wz2k

Labels
Good to fix PE-D severity.VeryLow Purely cosmetic and does not affect usage type.Documentation Improvements or additions to documentation
Projects
None yet
Milestone
v1.4
Development

Successfully merging a pull request may close this issue.

Update ug with PED fixes wz2k/tp
3 participants
@Zxun2 @wz2k @soc-se-bot