Skip to content

Conversation

@zpChris
Copy link
Contributor

@zpChris zpChris commented Jun 27, 2021

List of Changes:

  • Update the "Create a problem" link on the "Verified Problems" page to the correct url: /problem/create.
  • Update problem to only be set initially once.
  • Update Back button to return to previous page only if one exists on the same site; otherwise, go to the dashboard.
  • Restrict the problem description line length to one line.

Notes:

  • The problem must be set only initially once because every time it is set, a new id is given for each test case; this means that the JSON.stringify check on the Back button will sometimes return that the problem was updated, even when it was not.
  • I found the Back button "POP" check at this Stack Overflow post.
  • An explanation / demonstration of the line length clamp is shown at this screencast (albeit on another page, but still relevant).

Screencast and Images:

Screencast of the Fix Link Bugs and Update Site Design

Update Description Length on Verified Problems
This image just shows one small portion of the PR.
Update Description Length on Verified Problems

@zpChris zpChris added frontend design Improvement to frontend design labels Jun 27, 2021
@zpChris
Copy link
Contributor Author

zpChris commented Jun 27, 2021

@zpChris zpChris requested a review from alankbi June 28, 2021 00:23
Copy link
Contributor

@alankbi alankbi left a comment

Choose a reason for hiding this comment

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

Looks good to me! Left one comment about potentially reusing the line camp component (nice find btw 🙂), but otherwise should be good to merge whenever ready

@alankbi
Copy link
Contributor

alankbi commented Jun 29, 2021

Also, lmk if you planned on setting the min-width on this or another PR and I can take a look at that as well (it's a small change though so prob would be fine with just merging that without additional review)

@zpChris
Copy link
Contributor Author

zpChris commented Jul 2, 2021

Also, lmk if you planned on setting the min-width on this or another PR and I can take a look at that as well (it's a small change though so prob would be fine with just merging that without additional review)

I'll likely do this on another PR with a list of other small changes 👍

@zpChris zpChris merged commit 5389da0 into main Jul 3, 2021
@zpChris zpChris deleted the LinkBugFixes branch July 3, 2021 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

design Improvement to frontend design frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants