Opened on 02/03/2015 at 11:35:27 AM

Last modified on 10/09/2019 at 06:22:23 PM

#1924 new defect

CSS syntax of filter is not validated in like Chrome

Reported by: passbrains Assignee:
Priority: P3 Milestone:
Module: Maxthon Keywords:
Cc: Blocked By:
Blocking: Platform: Maxthon
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by Ross)

Adapted from ​https://www.passbrains.com/dashboard/view-ticket.php?ticket_no=MAX-55

Environment

Windows + 8 64bit + Maxthon + English

How to reproduce

  1. Open www.timesofindia.com
  2. Filter rule is 

indiatimes.com##div[style="float:left;font-size:20px;font-weight:normal;font-family:georgia;padding:10px;"]
Add this filter without first quotes for ex: 
indiatimes.com##div[style=float:left;font-size:20px;font-weight:normal;font-family:georgia;padding:10px;"]

Chrome displays filter rule "is not a valid CSS selector".

Observed behaviour

Maxthon accepts filter without any error.

Expected behaviour

Maxthon should display filter rule "is not a valid CSS selector".

Attachments (2)

3766_1422806800_maxthonCss.png​ (469.2 KB) - added by passbrains on 02/03/2015 at 11:35:32 AM.
3766_1422806800_chromeCss.png​ (198.3 KB) - added by passbrains on 02/03/2015 at 11:35:34 AM.

Download all attachments as: .zip

Change History (6)

Changed on 02/03/2015 at 11:35:32 AM by passbrains

Changed on 02/03/2015 at 11:35:34 AM by passbrains

comment:1 Changed on 02/18/2015 at 02:49:16 PM by Ross

  • Description modified (diff)
  • Summary changed from CSS syntax of filter is not validated. to CSS syntax of filter is not validated in like Chrome

comment:2 Changed on 03/05/2015 at 12:46:23 PM by jobp

We've looked into this issue, and it doesn't seem to effect other invalid selectors break element hiding. We added the following filters to ABP for Maxthon, and visited adblockplus.org to see whether the site rendered empty:
adblockplus.org##*[style="]
adblockplus.org##body

It did. Therefore, prioritized as P3 for now.

comment:3 Changed on 03/05/2015 at 12:46:34 PM by jobp

  • Priority changed from Unknown to P3

comment:4 Changed on 10/09/2019 at 06:22:23 PM by greiner

  • Component changed from Unknown to Maxthon
  • Tester set to Unknown
  • Verified working unset

Add Comment

Modify Ticket

Change Properties
Action
as new .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from (none).
Next status will be 'reviewing'.
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.