Skip to content

Move getDefaultDeviceName into the Platforms - #2643

Merged
dbkr merged 1 commit into
developfrom
dbkr/device_name_to_platform
Nov 24, 2016
Merged

Move getDefaultDeviceName into the Platforms#2643
dbkr merged 1 commit into
developfrom
dbkr/device_name_to_platform

Conversation

@dbkr

@dbkr dbkr commented Nov 24, 2016

Copy link
Copy Markdown
Member

So we can have a sensible device name on Electron

So we can have a sensible device name on Electron

@richvdh richvdh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm otherwise

console.log(window.process);
switch (window.process.platform) {
case 'darwin':
return 'macOS';

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

MacOS?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Nope: http://www.apple.com/uk/macos/sierra/ #branding #lowercase #flatdesign

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

#bollocks

@richvdh richvdh assigned dbkr and unassigned richvdh Nov 24, 2016
@dbkr
dbkr merged commit ac2911c into develop Nov 24, 2016
@dbkr
dbkr deleted the dbkr/device_name_to_platform branch December 14, 2016 18:09
t3chguy pushed a commit that referenced this pull request Mar 25, 2026
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants