Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

3824 - ignore =, <, >, +, -, *, / tokens#3962

Merged
jasonLaster merged 1 commit intofirefox-devtools:masterfrom
Fischer-L:3824-ignore-operator-tokens
Sep 10, 2017
Merged

3824 - ignore =, <, >, +, -, *, / tokens#3962
jasonLaster merged 1 commit intofirefox-devtools:masterfrom
Fischer-L:3824-ignore-operator-tokens

Conversation

@Fischer-L
Copy link
Contributor

Associated Issue: #3824

Summary of Changes

  • Add =, <, >, +, -, *, / tokens as invalid tokens when doing updatePreview

Test Plan

  1. Visit any website with codes using =, <, >, +, -, *, / tokens
  2. Pause on lines having the above invalid tokens
  3. Hover mouse over the above invalid tokens and make sure no preview for them triggered

Screenshots/Videos

Please see [1] for the screenshots gif
[1] #3824 (comment)

@codecov
Copy link

codecov bot commented Sep 10, 2017

Codecov Report

Merging #3962 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3962   +/-   ##
=======================================
  Coverage   55.68%   55.68%           
=======================================
  Files         128      128           
  Lines        4895     4895           
  Branches     1014     1014           
=======================================
  Hits         2726     2726           
  Misses       2169     2169
Impacted Files Coverage Δ
src/utils/editor/expression.js 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 09438b2...a9990a1. Read the comment docs.

@Fischer-L
Copy link
Contributor Author

Fischer-L commented Sep 10, 2017

@jasonLaster Hi, how could I set a reviewer? I couldn't see a Reviewers option when/after creating PR. (the right side Reviewers section is unclickable but just plain test area)Thanks.

@jasonLaster
Copy link
Contributor

@Fischer-L happy to look.

Copy link
Contributor

@jasonLaster jasonLaster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jasonLaster jasonLaster merged commit 838f0b1 into firefox-devtools:master Sep 10, 2017
@jasonLaster
Copy link
Contributor

Have you joined our slack? https://devtools-html-slack.herokuapp.com

@Fischer-L
Copy link
Contributor Author

Thanks joined

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants