Closed
Bug 1035780
Opened 10 years ago
Closed 10 years ago
Submit button remains stuck when "Quote text is required" error is displayed
Categories
(Marketplace Graveyard :: Curation Tools, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: julian.mozbugs, Assigned: spasovski)
References
Details
(Keywords: regression)
Steps to reproduce: 1. Load Create a Featured app landing page https://marketplace-dev.allizom.org/curate/create/apps 2. Select an app from the App selector and add a slug. 3. Select rating stars from the Quote section (don't add Quote text). 4. Add Description text to the Description section. Expected results: "Quote text is required" error is displayed, the user is able to correct the errors and submit the featured app again. Actual results: The "Submit" button changes its label to "Submitting..." and it remains stuck like that, even after the user corrects the errors. Notes/Issues: Verified on FF33 (Win 7). Screencast for this issue http://screencast.com/t/l1JoOSbmQHn
Reporter | ||
Comment 1•10 years ago
|
||
I just noticed that this issue is reproducing for every displayed front-end validation error (App, Slug or Quote is required)
Updated•10 years ago
|
Assignee: nobody → dspasovski
Priority: -- → P2
Assignee | ||
Comment 2•10 years ago
|
||
https://github.com/mozilla/transonic/compare/b13d754d8fe8...8b2ce04afe93 This handles the case of the button not being clickable again. Incomplete but required fields on other tabs will still generate errors so it's better to clear those before submitting again.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 3•10 years ago
|
||
Issue is still reproducing in https://marketplace-dev.allizom.org/curate/create/apps on FF33 (Win 7). Please see screencast http://screencast.com/t/CGOv8Ri28Ofy
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 4•10 years ago
|
||
https://github.com/spasovski/transonic/commit/e07cc8d6ada2bfba284c008546d87ea98ec7a756 That was a very good find. Thanks.
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 5•10 years ago
|
||
Verified as fixed in https://marketplace-dev.allizom.org/curate/ on FF33 (Win 7). Postfix screencast http://screencast.com/t/kieQSq1Zi2H Closing bug.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•