Page MenuHomePhabricator

Tokenizers fail in Firefox on Android
Open, WishlistPublic

Description

Attempting to add a project to a task in Maniphest fails on Firefox 30 on Android.

Reproduction

  1. Create a new task and attempt to add a project / CC / or Assignee in Firefox on Android.
  2. Type a username or project into their respective areas and tap outside the area, the content inside the box with disappear.
  • The same thing happens when using the Comment, etc section.

Bonus Points

  1. When adding a project / CC / or Assignee, no tooltip/suggestion box appears.

Event Timeline

Yomi raised the priority of this task from to Needs Triage.
Yomi updated the task description. (Show Details)
Yomi added a subscriber: Yomi.
Yomi updated the task description. (Show Details)
Yomi renamed this task from Adding / Associating a Project/Assignee/CC to a project fails in Firefox on Android to Adding / Associating a Project /Assignee / CC to a project fails in Firefox on Android.Jun 16 2014, 1:19 AM
chad triaged this task as Wishlist priority.Jun 16 2014, 5:41 AM
chad added a subscriber: chad.

These browsers are likely too edge case for us, and they're not "officially" supported. However, we're happy to take patches for them.

epriestley renamed this task from Adding / Associating a Project /Assignee / CC to a project fails in Firefox on Android to Tokenizers fail in Firefox on Android.Sep 8 2014, 5:11 PM
epriestley added a project: Android.
isaura_84 raised the priority of this task from Wishlist to Normal.May 26 2015, 1:33 PM
isaura_84 raised the priority of this task from Normal to Unbreak Now!.
epriestley lowered the priority of this task from Unbreak Now! to Wishlist.May 26 2015, 1:36 PM
epriestley added a subscriber: epriestley.
lemwill moved this task from tak ak tak to Backlog on the Android board.
lemwill moved this task from Backlog to tak ak tak on the Android board.
lemwill moved this task from tak ak tak to dd on the Android board.
filajacek moved this task from tak ak tak to Backlog on the Android board.
filajacek moved this task from Backlog to dd on the Android board.
haldyr moved this task from Testovaci collumn to Backlog on the Android board.
tputnoky moved this task from Icebox to Backlog on the Android board.
griggsca91 moved this task from Icebox to Backlog on the Android board.
cyn0228.ch added revisions: D14011: Web Push notifications, D13984: [drydock/core] Support custom attributes on all blueprints, D13909: Convert DifferentialDiff view to new PHUIDiffTableOfContentsListView, D13350: Show internal clone URIs for external repositories, D13424: Add a method for determining the relative path, D13421: Prevent rapid AWS calls being made due to SSH test returning quickly, D13373: Separate interfaces in library maps, D13354: First of a two-part fix where the availability wasn't taking into account all-day and recurring events., D14065: Remove some unused methods from ArcanistLinter, D14066: Update "should not run as root" message, D14067: Drop the `metamta_mailinglist` table, D14068: Mark `PhabricatorPHIDType::getPHIDTypeApplicationClass()` as abstract, D14069: Fix some failing unit tests, D14074: Add escaping for Checkstyle output, D14073: Various translation improvements, D14072: Various translation improvements, D14071: Various translation improvements, D14070: Add a `phutil_count` function, D14059: Build LauncherButton for PHUIObjectItemView, D14061: Correct a bad Herald migration, D14063: Fix word-break on remarkup tables, D14064: Add dialog to purge opcode/data caches, D14057: Modularize generation of supplemental login messages, D14049: Modularize Aphront exception handling, D14050: Modernize OAuthserver and provide more context on "no permission" exception, D14053: Fix callsites which called libphutil_console_wrap like it were _format, D14054: Shuffle various parts of the config edit UI around, D14055: Add support for temporary files to `file.allocate`, D14056: Add temporary file support to ArcanistFileUploader, D14034: Prevent caching of workflows, D14035: update libphutil (see merge commit), D14036: Scroll on overflow of property list (mobile), D14037: Allow unit test engines to render their own results, D14042: Add CCs to Phriction Edit page, D14043: Minor Ponder Comment tweaks, D14044: Fix abrupt failure mode for uncloned repositories in Diffusion, D14045: Add a "Printable Version" link to Phortune invoices, D14046: Fix double period in PhutilMissingSymbolException, D14047: Replace AphrontUsageException with AphrontMalformedRequestException, D14033: staging repo compatibility for older git versions, D14032: Allow Controllers to return a wider range of "response-like" objects, D14031: Fix XHPAST Makefile on OS X, D14030: Remove errant in arcanist.php, D14029: Fix an issue with "packages(...)" in typeaheads, D14028: Improve XHPAST handling of syntax errors, D14027: Delete another problematic XML test file, D14026: Use phutil_hashes_are_identical() when comparing hashes in Phabricator, D14025: Add a constant-time string comparison function to defuse timing and type-juggling attacks, D14024: Fix unsafe string for static method calls, D14023: Update Owners docs a bit, D14022: Fix an issue where paths could bleed across repos in Owners, D14021: Don't copy `null` attributes passed to JX.$N(), D14020: Update bar colors to match icon colors, D14019: Quickly fix phpqrcode syntax, D14000: Fix diviner symbols documentation, D14001: Update Releeph for handleRequest, D14002: Update Conpherence layout for logged out view, D14003: Basic Answer Wiki for Ponder, D14004: Make PHP echo tags always available, D14005: Add some missing translations, D14010: Test XHPAST linter rules in isolation, D14008: Push construction of routing maps into Sites, D14007: Swap duplicate close status on Ponder for invalid, D14006: Allow searching for files by name, D13999: Fix possible recursive embeds in Dashboard text panels, D13998: Use 'remote.origin.url' fallback for git < 1.7.5, D13997: Prevent "commit message magic words" parser from exploding on "reverts aaaa.....aaz", D13996: Remove some unused includes, D13995: Remove unused file, D13994: Minor tidying of parser_nodes script, D13993: Use 'git blame --porcelain' for git blame info, D13987: Minor tidying of XHPAST code, D13988: Add very basic routing to Nuance, D13989: Minimize reliance on 'git branch' output format, D13991: Recover from PyLint raising messages at character "-1", D13992: Remove two problematic XML linter unit tests, D13990: Fix mail parameter error with old migrations, D13986: Remove unused last_token macro, D13985: [drydock/core] Allow execution timeout to be set when running commands, D13975: Remove include_debug from yy_extra_type, D13976: Remove unused replacestr function from XHPAST, D13977: Minify XHPAST output, D13978: Improvements to makefile, D13979: Make XHPAST scanner non-interactive, D13980: Add basic "View" and "Edit" features to Nuance, D13981: Use didRejectResult() when querying workboard columns, D13982: Tune document details in Legalpad, D13983: Avoid parsing git "remote show" using "ls-remote", D13965: Add a main page to Nuance, D13966: Allow Nuance source definitions to add actions to source views, D13968: Improve XHPAST handling of traits, D13967: Improve handling of PHP 5.4 features, D13969: Remove flex and bison debug flags, D13970: Enable bison warnings, D13971: Fix bison "empty" warnings, D13972: Fix a bison deprecation warning, D13973: Remove some unused fields from yy_extra_type, D13974: Fix bison warnings.Sep 7 2015, 10:42 PM
chad removed revisions: D13974: Fix bison warnings, D13973: Remove some unused fields from yy_extra_type, D13972: Fix a bison deprecation warning, D13971: Fix bison "empty" warnings, D13970: Enable bison warnings, D13969: Remove flex and bison debug flags, D13967: Improve handling of PHP 5.4 features, D13968: Improve XHPAST handling of traits, D13966: Allow Nuance source definitions to add actions to source views, D13965: Add a main page to Nuance, D13983: Avoid parsing git "remote show" using "ls-remote", D13982: Tune document details in Legalpad, D13981: Use didRejectResult() when querying workboard columns, D13980: Add basic "View" and "Edit" features to Nuance, D13979: Make XHPAST scanner non-interactive, D13978: Improvements to makefile, D13977: Minify XHPAST output, D13976: Remove unused replacestr function from XHPAST, D13975: Remove include_debug from yy_extra_type, D13985: [drydock/core] Allow execution timeout to be set when running commands, D13986: Remove unused last_token macro, D13990: Fix mail parameter error with old migrations, D13992: Remove two problematic XML linter unit tests, D13991: Recover from PyLint raising messages at character "-1", D13989: Minimize reliance on 'git branch' output format, D13988: Add very basic routing to Nuance, D13987: Minor tidying of XHPAST code, D13993: Use 'git blame --porcelain' for git blame info, D13994: Minor tidying of parser_nodes script, D13995: Remove unused file, D13996: Remove some unused includes, D13997: Prevent "commit message magic words" parser from exploding on "reverts aaaa.....aaz", D13998: Use 'remote.origin.url' fallback for git < 1.7.5, D13999: Fix possible recursive embeds in Dashboard text panels, D14006: Allow searching for files by name, D14007: Swap duplicate close status on Ponder for invalid, D14008: Push construction of routing maps into Sites, D14010: Test XHPAST linter rules in isolation, D14005: Add some missing translations, D14004: Make PHP echo tags always available, D14003: Basic Answer Wiki for Ponder, D14002: Update Conpherence layout for logged out view, D14001: Update Releeph for handleRequest, D14000: Fix diviner symbols documentation, D14019: Quickly fix phpqrcode syntax, D14020: Update bar colors to match icon colors, D14021: Don't copy `null` attributes passed to JX.$N(), D14022: Fix an issue where paths could bleed across repos in Owners, D14023: Update Owners docs a bit, D14024: Fix unsafe string for static method calls, D14025: Add a constant-time string comparison function to defuse timing and type-juggling attacks, D14026: Use phutil_hashes_are_identical() when comparing hashes in Phabricator, D14027: Delete another problematic XML test file, D14028: Improve XHPAST handling of syntax errors, D14029: Fix an issue with "packages(...)" in typeaheads, D14030: Remove errant in arcanist.php, D14031: Fix XHPAST Makefile on OS X, D14032: Allow Controllers to return a wider range of "response-like" objects, D14033: staging repo compatibility for older git versions, D14047: Replace AphrontUsageException with AphrontMalformedRequestException, D14046: Fix double period in PhutilMissingSymbolException, D14045: Add a "Printable Version" link to Phortune invoices, D14044: Fix abrupt failure mode for uncloned repositories in Diffusion, D14043: Minor Ponder Comment tweaks, D14042: Add CCs to Phriction Edit page, D14037: Allow unit test engines to render their own results, D14036: Scroll on overflow of property list (mobile), D14035: update libphutil (see merge commit), D14034: Prevent caching of workflows, D14056: Add temporary file support to ArcanistFileUploader, D14055: Add support for temporary files to `file.allocate`, D14054: Shuffle various parts of the config edit UI around, D14053: Fix callsites which called libphutil_console_wrap like it were _format, D14050: Modernize OAuthserver and provide more context on "no permission" exception, D14049: Modularize Aphront exception handling, D14057: Modularize generation of supplemental login messages, D14064: Add dialog to purge opcode/data caches, D14063: Fix word-break on remarkup tables, D14061: Correct a bad Herald migration, D14059: Build LauncherButton for PHUIObjectItemView, D14070: Add a `phutil_count` function, D14071: Various translation improvements, D14072: Various translation improvements, D14073: Various translation improvements, D14074: Add escaping for Checkstyle output, D14069: Fix some failing unit tests, D14068: Mark `PhabricatorPHIDType::getPHIDTypeApplicationClass()` as abstract, D14067: Drop the `metamta_mailinglist` table, D14066: Update "should not run as root" message, D14065: Remove some unused methods from ArcanistLinter, D13354: First of a two-part fix where the availability wasn't taking into account all-day and recurring events., D13373: Separate interfaces in library maps, D13421: Prevent rapid AWS calls being made due to SSH test returning quickly, D13424: Add a method for determining the relative path, D13350: Show internal clone URIs for external repositories, D13909: Convert DifferentialDiff view to new PHUIDiffTableOfContentsListView, D13984: [drydock/core] Support custom attributes on all blueprints, D14011: Web Push notifications.Sep 7 2015, 10:45 PM
developerlin moved this task from Icebox to Backlog on the Android board.
cyen raised the priority of this task from Wishlist to Needs Triage.Dec 9 2015, 9:54 AM
cyen triaged this task as Wishlist priority.

I can still reproduce this on Firefox 49.0.2 , using the default Samsung keyboard on my tab.
However, if I switch to the Hacker's Keyboard, it works correctly.

tranba moved this task from Backlog to Icebox on the Android board.