-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
chore: sort the frameworks select dropdown in a more logical way #21553
Conversation
Thanks for taking the time to open a PR!
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this new order. Only comment is: If we ever need to change order again in the future, it could be made easier in the code.
I would prefer a manually set array of keys, clearer and more explicit.
It would be easier to adjust if we need to.
@lmiller1990 Can you create a corresponding Jira issue to track this work? I would prefer just alphabetical as an aside. If this screenshot reflects the current state, it still looks illogical to me. |
@jennifer-shehane done, also just changed to alphabetical order, much more simple + added tests. |
@lmiller1990 which screenshot is the source of truth? Looks like neither screenshot is in alphabetical order, so I feel like i'm missing something. |
24a975c
to
ff425a6
Compare
@emilyrohrbough oops, I've updated the screenshot. thanks! |
…pack * develop: test(launchpad): skip failure due to recaptcha on windows, enable more windows jobs (#21620) docs: add "Steps to test" to PR template (#21606) feat: (origin) handle waiting for aliased intercepts (#21579) fix: handle git watch error (#21600) fix: Successfully create new specs files that do not have a known extension (#21593) chore: release 9.7.0 fix: do not allow experimentalSessionAndOrigin to be available in CT … (#21588) chore: sort the frameworks select dropdown in a more logical way (#21553) Trigger Build fix: disable ct testing when run via binary (#21540) chore: Update Chrome (beta) to 102.0.5005.61 (#21556) chore: release @cypress/react-v5.12.5 fix: add support for Next.js v12.1.6 (#21516) chore: release @cypress/schematic-v1.7.0 feat(cypress/schematic): add headed option and other fixes (#21415) chore: release @cypress/vite-dev-server-v2.2.3 fix: handle specs with white space in vite-dev-server (#21386) Empty commit
User facing changelog
Sort the frameworks in a more structured manner on the select framework screen.
Additional details
Put in alphabetical order.
Open to other ways to order the list, just threw this together since the current list is just in an arbitrary order.
How has the user experience changed?
PR Tasks
cypress-documentation
?type definitions
?cypress.schema.json
?