-
Notifications
You must be signed in to change notification settings - Fork 203
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
Greatest hits radio select #432
Comments
Rephrasing @miketaylr (playing 🐦).
|
|
Let's do that and if it's not working well. we will tweak again. Step by step. day by day. :) |
This looks great, but we probably still want a textarea if people want to write in some more detail. Or maybe have a button that says "Give more details" or something that shows a hidden textarea? |
Let me just agree with both last speakers.. |
Ignore the font change, I used a screenshot. Here is the text field back. I think we should leave the field in and expanded, because we'll get more comments that way. We could say "Additional comments" instead of Give more details, or just Comments. A button to open the field competes with the button to submit and the link (as shown here) may be confusing or just not inspire action. |
Looks great @calexity. |
Looking at @magsout's commit, I wonder what about the case where the issue is "none of the above"? The current proposed design doesn't give an option for a user to write in their own summary. Would an option like "None of these, I'll write my own" that shows the "Problem in 5 words*" input be possible? |
@miketaylr @magsout I think that's fine, what about just moving "Add more details" up to right below this to make it more clear that they should just add it there? |
That could work @calexity. Would be cool to see a small mockup when you get a chance. Would just need to come up with a nice generic bug title "bloop.com - [something generic here]" |
Cool. I just realized "Don't know but something wrong" can probably be replaced entirely with "Other - I'll add details below." |
That sounds good. Maybe we take the first few words from the description if it exists, and use that as a default. |
@miketaylr my PR it' ok with proposal from @calexity ? Just need to manage |
/o, sorry for not looking at this for a while @magsout. I'll check out that PR either at some point today or tomorrow. It would be good to finish this feature soon to be able to tackle image uploading. |
OK, for people not following #524, I screwed up rebasing the branch and ended up closing the PR. Let me see if I can get this guy working and we'll pick it up from there. edit: issues/432/2 is the branch we can work on. |
Here's what I see @magsout: https://cldup.com/ZxLn-ZfVnM-2000x2000.png Looks good to me. So to finish the first parts of this bug (minus image upload etc.), there's two parts for me:
Looking at the form with this patch applied on desktop, I think we could just swap the position of the columns putting the inputs on the left and the radios on the right and then we wouldn't need to try any form factor detection.
|
Alright, let's start with these:
|
Making a TODO list:
|
(and use named arguments in the description of build_formdata)
OK, I think we should probably make the radio choices required so I went ahead and did that. I deployed everything so far to staging.webcompat.com if people wanna check it out. @magsout any CSS tweaks are most welcome. 😄 |
Oops, broke some tests. But I need to update tests for radio validation anyways. |
@miketaylr late to the party, but yes, I'd make it required. All these fields are required. In fact, we could say that at the top or bottom "All fields are required so we can fix the problem." |
@calexity cool, I made it required. |
Follow up to #432 - CSS refactor, update tests
* master: (46 commits) Fixes #509 - mention [ci skip] in docs. v1.6.2 changelog Issues #588 - list style Fix #583 Using the function for checking dependencies Checking dependencies Modules needed for checking dependencies Two cases to handle Adding message Issue #368 - Remove (now) unused STARTUP variable Issue #368 - Compute cache busting param based on md5 hash of file Issue #368 - Move bust_cache function into a Jinja template helper Issue #368: Part 1, add bust_cache param to minified production JS Issue #587 - Add a task to check npm dependencies and run by default v1.6.1 Fixes #591 - Typo in form field. >_< Tag v1.6.0 Issue #572 - Remove right float for GitHub issue warp hint Issue #432 - Add js-form-error and js-no-error classes and update tests. Issue #432 - Add wc-Form-required class to problem label Added wc-Form, wc-ReportForm ...
For bug reporting, populate a short list of most-submitted issues so it's easier to select what's going wrong.
As a mobile bug reporter
When I report a bug
Then I want it to be easier to select what's wrong
So I can report it faster.
The text was updated successfully, but these errors were encountered: