Skip to content
This repository has been archived by the owner on Nov 22, 2021. It is now read-only.

Fix touchbar and add repost button to it. #233

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

Impakt
Copy link

@Impakt Impakt commented May 13, 2021

Touch Bar Shot 2021-05-14 at 12 44 17 pm

soundcloud.repost()
}
})

Copy link
Author

Choose a reason for hiding this comment

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

The original PR included a reposted icon but no code associated with it. If there was a way to detect if a track has already been reposted we can update the icon, but i'm not sure how to do that.

new TouchBarSpacer({ size: 'flexible' })
])
const touchBar = new TouchBar({
items:[
Copy link
Author

Choose a reason for hiding this comment

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

this was the reason why the touchbar wasn't working earlier

@Impakt Impakt changed the title Fixed touchbar and added repost button to it. Fix context menu, add open in browser menu item, fixed touchbar and add repost button to it. May 13, 2021
@Impakt Impakt changed the title Fix context menu, add open in browser menu item, fixed touchbar and add repost button to it. Fix context menu, add open in browser menu item, fix touchbar and add repost button to it. May 13, 2021
@Impakt Impakt changed the title Fix context menu, add open in browser menu item, fix touchbar and add repost button to it. Fix touchbar and add repost button to it. May 14, 2021
@Impakt
Copy link
Author

Impakt commented May 14, 2021

@rmaes4 this may interest you

soundcloud.on('play', ({ title, subtitle }) => {
soundcloud.on('play-new-track', ({ title, subtitle, artworkURL }) => {
let displayTitle = `${title} by ${subtitle}`
displayTitle = displayTitle.padEnd(displayTitle.length * 1.3, ' ')
Copy link
Author

Choose a reason for hiding this comment

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

this seems hacky but it was the only way to not have the string truncated in the touch bar. You can now swipe on the scrubber to see the whole string

});
res.on('end', () => {
titleScrubber.items = [{
label:''
Copy link
Author

Choose a reason for hiding this comment

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

this stops the artwork from having rounded corners on the left hand side

@rmaes4
Copy link
Contributor

rmaes4 commented May 14, 2021

@Impakt Awesome! I am quite busy these days and haven't worked on this project in a long time but it is great to see more progress.

@sonarcloud
Copy link

sonarcloud bot commented Jun 10, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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