Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

The add() method of Set returns wrong values in webpage IDE console panel #5493

Closed
3 tasks done
akasuv opened this issue Feb 19, 2021 · 3 comments
Closed
3 tasks done

Comments

@akasuv
Copy link

akasuv commented Feb 19, 2021

馃悰 bug report

Preflight Checklist

  • I have read the
    Contributing Guidelines
    for this project.
  • I agree to follow the
    Code of Conduct
    that this project adheres to.
  • I have searched the issue tracker for an issue that matches the one I want
    to file, without success.

Description of the problem

I was reading JavaScript: The Definitive Guide, and trying some Set methods with CodeSandbox, and found the add() method returns different values in IDE console panel and Chrome build-in console.

The add() method takes a single argument; if you pass an array, it adds the array itself to the set, not the individual array elements. add() always returns the set it is invoked on

IDE console

image

Chrome built-in console

image

How has this issue affected you? What are you trying to accomplish?

I think it's a language implementation level problem.

To Reproduce

Check the link below

Link to sandbox: link

Your Environment

Software Name/Version
小odesandbox Webpage
Browser Version 90.0.4422.0 (Official Build) canary (x86_64)
Operating System macOS Big Sur Version 11.2.1
@akasuv akasuv changed the title The add() method of Set returns wrong value in webpage IDE console panel The add() method of Set returns wrong values in webpage IDE console panel Feb 19, 2021
@github-actions
Copy link

This issue has automatically been marked stale because there has been no activity in a while. Please leave a comment if the issue has not been resolved, or if it is not stale for any other reason. After 2 weeks, this issue will automatically be closed, unless a comment is made or the stale label is removed.

@lbogdan
Copy link
Contributor

lbogdan commented May 24, 2021

Hey @akasuv ,

Thanks for your report!

The linked sandbox no longer exists, but I replicated this in https://codesandbox.io/s/console-feed-set-issue-z7p2k and will open an issue against the upstream dependency, console-feed.

@github-actions
Copy link

This issue has automatically been marked stale because there has been no activity in a while. Please leave a comment if the issue has not been resolved, or if it is not stale for any other reason. After 2 weeks, this issue will automatically be closed, unless a comment is made or the stale label is removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants