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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Samsung galaxy tab s8 and s9 devices are not recognised #740

Open
JDWaghela opened this issue Aug 16, 2024 · 5 comments
Open

Samsung galaxy tab s8 and s9 devices are not recognised #740

JDWaghela opened this issue Aug 16, 2024 · 5 comments

Comments

@JDWaghela
Copy link

Library version
0.7.30

Describe the bug
Samsung Galaxy Tab s8 and s9 are returning empty object in below code
const { device } = UAParser();
Therefore we're not able to identified that these devices are tablet, and instead it is behaving like desktop mode.

To Reproduce
Steps to reproduce the behavior:

const { device } = UAParser();
console.log(device); // resulting output is {}

Expected behavior
It should return device type as tablet

Screenshots
image

Smartphone (please complete the following information):

  • Device: Samsung Galaxy Tablet s8 & s9
  • OS: Android
  • Browser : Chrome
@faisalman
Copy link
Owner

Hi, can I get a confirmation of the detection result from https://uaparser.dev/?

@JDWaghela
Copy link
Author

https://uaparser.dev/

@faisalman Please find below response from that site in tablets

Samsung Galaxy Tablet S8

{
  "ua": "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/123.0.0.0 Safari/537.36",
  "browser": {
    "name": "Chrome",
    "version": "123.0.6312.40",
    "major": "123"
  },
  "cpu": {
    "architecture": "amd64"
  },
  "device": {},
  "engine": {
    "name": "Blink",
    "version": "123.0.0.0"
  },
  "os": {
    "name": "Linux",
    "version": ""
  }
}

Samsung Galaxy Tablet S9

{
  "ua": "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/127.0.0.0 Safari/537.36",
  "browser": {
    "name": "Chrome",
    "version": "127.0.6533.64",
    "major": "127"
  },
  "cpu": {
    "architecture": "amd64"
  },
  "device": {},
  "engine": {
    "name": "Blink",
    "version": "127.0.0.0"
  },
  "os": {
    "name": "Linux",
    "version": ""
  }
}

@developersoom
Copy link

Hello @faisalman, A similar problem has occurred with Galaxy Note 9. Could you please check as well?

The result from https://uaparser.dev is as below.

{
  "ua": "Mozilla/5.0 (Linux; Android 10; K) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/127.0.0.0 Mobile Safari/537.36",
  "browser": {
    "name": "Chrome",
    "version": "127.0.6533.103",
    "major": "127"
  },
  "cpu": {},
  "device": {
    "type": "mobile",
    "model": "SM-N960N"
  },
  "engine": {
    "name": "Blink",
    "version": "127.0.0.0"
  },
  "os": {
    "name": "Android",
    "version": "10.0.0"
  }
}

But using ua-parse-js(1.0.38), the result is as below(device is empty).

image

@mastrobardo
Copy link

I'm trying to use the library in our project, and this is exact issue we facing. Seems vendors are trying to market these tablets as desktop replacement. "linux"iswhat is really upsetting me

@mastrobardo
Copy link

mastrobardo commented Sep 23, 2024

Hello @faisalman, A similar problem has occurred with Galaxy Note 9. Could you please check as well?

The result from https://uaparser.dev is as below.

{
  "ua": "Mozilla/5.0 (Linux; Android 10; K) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/127.0.0.0 Mobile Safari/537.36",
  "browser": {
    "name": "Chrome",
    "version": "127.0.6533.103",
    "major": "127"
  },
  "cpu": {},
  "device": {
    "type": "mobile",
    "model": "SM-N960N"
  },
  "engine": {
    "name": "Blink",
    "version": "127.0.0.0"
  },
  "os": {
    "name": "Android",
    "version": "10.0.0"
  }
}

But using ua-parse-js(1.0.38), the result is as below(device is empty).

image

image
galaxy s9 on browserstack. android is never cited. Even worst, we dont know the model

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

5 participants
@mastrobardo @faisalman @JDWaghela @developersoom and others