Number field not rendered properly, I add that to the _bootstrap_process_input function

CommentFileSizeAuthor
#2 numberfield-2202153-0.patch355 bytesMajdi
numberfield-2145515-0.patch355 bytesMajdi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, numberfield-2145515-0.patch, failed testing.

Majdi’s picture

FileSize
355 bytes
csakiistvan’s picture

Version: 7.x-3.0 » 7.x-3.x-dev

The patch is not works 3.0, its made to 3.x-dev I think (in 3.0, _bootstrap_process_input missing Webform rows).

I think the test fallen for that.

csakiistvan’s picture

Status: Needs work » Needs review
csakiistvan’s picture

2: numberfield-2202153-0.patch queued for re-testing.

csakiistvan’s picture

It works for me, before and after screenshots.

Majdi’s picture

cool,
I check all webform fields, This is the only missing field, I just test it again on fresh installation and it works fine

markhalliwell’s picture

Priority: Major » Normal
Status: Needs review » Fixed
Issue tags: -numberfield

Thanks @Majdi!

Committed 672d3e8 to 7.x-3.x:

Issue #2202153 by Majdi: Webform number field not rendered properly.

markhalliwell’s picture

Version: 7.x-3.x-dev » 8.x-3.x-dev
Assigned: Unassigned » ryan.armstrong
Category: Bug report » Feature request
Status: Fixed » Needs review

  • Commit 672d3e8 on 7.x-3.x, 8.x-3.x authored by Majdi, committed by Mark Carver:
    Issue #2202153 by Majdi: Webform number field not rendered properly.
    
m1r1k’s picture

Assigned: ryan.armstrong » Unassigned
Status: Needs review » Postponed

Webform doesn't have stable 8.x port yet

  • Mark Carver committed 672d3e8 on 8.x-3.x.x authored by Majdi
    Issue #2202153 by Majdi: Webform number field not rendered properly.
    
markhalliwell’s picture

Version: 8.x-3.x-dev » 7.x-3.x-dev
Category: Feature request » Bug report
Status: Postponed » Closed (fixed)

I'm just moving this back to 7.x. We'll re-evaluate 8.x and create new issues one we get to that point.