GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Add tutorial: setting-up Bluetooth - PulseAudio by BertLindeman · Pull Request #24 · ev3dev/ev3dev.github.io

Conversation

@BertLindeman

@BertLindeman BertLindeman commented Nov 17, 2014

Copy link
Copy Markdown
Contributor

Probably needs more work, but this is a good stage to ask for comments.
On Windows the tutorials page did not look OK:
tutorial_on_win7

But that could well be a jekyll on win artefact.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
BertLindeman added 3 commits November 17, 2014 20:55
@BertLindeman
Add Bluetooth tutorial
271d3ff 
Formatting still needs work
@BertLindeman
Update and rename bluetooth-audio-with-pulseaudio.md to setup-bluetoo…
50787e9 
…th-audio-with-pulseaudio.md

improve format
@BertLindeman
Update and rename setup-bluetooth-audio-with-pulseaudio.md to setting…
e18a9f1 
…-up-bluetooth-audio-with-pulseaudio.md

rename to conform with the existing tutorials
@dlech

dlech commented Nov 18, 2014

Copy link
Copy Markdown
Member

But that could well be a jekyll on win artifact.

If you have not recently, run bundle update. Also, it should be serving over http://, not file://
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
dlech
dlech reviewed Nov 18, 2014
View reviewed changes
Comment thread docs/tutorials/setting-up-bluetooth-audio-with-pulseaudio.md Outdated

@dlech dlech Nov 18, 2014

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Leading spaces here are causing line break problems.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@dlech

dlech commented Nov 18, 2014

Copy link
Copy Markdown
Member

This is not very readable as-is, so I will let you clean it up a bit based on my comments and then have a look at it again.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@BertLindeman
Add Bluetooth tutorial
5565cd7 
First markdown cleanup round
Updated titles to conform to existing tutorials
Format of "pactl list cards" output is horrible.
@BertLindeman

BertLindeman commented Nov 18, 2014

Copy link
Copy Markdown
Contributor Author

I assUme commit 5565cd7 is included as the diffs above are marked "outdated"

The format of the output of 'pactl list cards' is horrible on windows
but in github OK, but the numbered list restarts after this output. It should not.
There is probably more formatting not completely correct.
Thanks David.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@dlech

dlech commented Nov 18, 2014

Copy link
Copy Markdown
Member

GitHub and GitHub pages use different markdown engines, so it is possible that there are some differences. If things are still looking horrible on windows, they probably will look horrible everywhere. Be sure you are following the instructions from here to preview on Windows. Also, you can create a gh-pages branch in your fork of this repo and see what it will actually look like on the web (should be the same as viewing locally if you are doing it right).
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
dlech
dlech reviewed Nov 18, 2014
View reviewed changes
Comment thread docs/tutorials/setting-up-bluetooth-audio-with-pulseaudio.md Outdated

@dlech dlech Nov 18, 2014

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Also, you need a blank line between the line that starts with 1. and the line with the "code".

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@dlech

dlech commented Nov 18, 2014

Copy link
Copy Markdown
Member

This should help: http://kramdown.gettalong.org/syntax.html
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
dlech
dlech reviewed Nov 18, 2014
View reviewed changes
Comment thread docs/tutorials/setting-up-bluetooth-audio-with-pulseaudio.md Outdated

@dlech dlech Nov 18, 2014

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

We should say that running PulseAudio provides these two profiles and that the device we are connecting to needs to have at least one of these profiles.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@WasabiFan

WasabiFan commented Nov 18, 2014

Copy link
Copy Markdown
Member

@BertLindeman

But that could well be a jekyll on win artefact.

From your screenshot, it doesn't look like you are looking at it from the server. The path in the address bar is a "file://" URI, which will just retrieve raw page content (without styles). Open the server (usually http://localhost:4000/) and it should work better, assuming you have the server running.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@BertLindeman
Add Bluetooth tutorial
ef8c418 
formatting
Changed bluetoothctl into brickman
@BertLindeman

BertLindeman commented Nov 18, 2014

Copy link
Copy Markdown
Contributor Author

@dlech @WasabiFan Thanks for all the tutorial and guiding.
Updated formatting
Changed bluetoothctl into brickman
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@BertLindeman @dlech @WasabiFan
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.