Skip to content

Lrrecords branding#55

Open
lrrecords wants to merge 2 commits intomontulli:masterfrom
lrrecords:lrrecords-branding
Open

Lrrecords branding#55
lrrecords wants to merge 2 commits intomontulli:masterfrom
lrrecords:lrrecords-branding

Conversation

@lrrecords
Copy link
Copy Markdown

Title
Add LRRecords-branded wrapper for GrooveScribe (lr-brand)

Body
Adds a new folder lr-brand with a branded wrapper page that embeds the GrooveScribe app in an iframe.

Files added:

  • lr-brand/index.html — branded wrapper page
  • lr-brand/css/lr-brand.css — brand stylesheet (uses the palette provided)
  • lr-brand/assets/lrrecords-logo.png — supplied logo
  • lr-brand/README_BRANDING.md — install & embed instructions

Deployment notes:

  • Option A (recommended): place lr-brand at /drumming-tutor/ on your site and ensure the iframe src in lr-brand/index.html points to the correct app entry:
    • If the app remains at the repo root, use iframe src="../index.html".
    • If you copy the app into lr-brand/app/, use iframe src="./app/index.html".
  • Option B: deploy lr-brand as a static site (Netlify/GitHub Pages/Vercel) and embed it in EasyFunnels via iframe.

How to swap the logo / change iframe paths:

  • Replace lr-brand/assets/lrrecords-logo.png with a different PNG or an SVG (same filename) and commit.
  • Edit the iframe src line in lr-brand/index.html to point at the desired app path.

Notes:

  • Hosting the wrapper and app on the same origin is recommended to avoid CORS/localStorage issues.
  • If you prefer, I can instead apply branding directly inside the GrooveScribe app (replace logos & CSS). That would change the app pages in-place — I can prepare a PR for that too if you want.

Embed snippet for EasyFunnels (copy into the embed widget)

  • If hosting wrapper at https://lrrecords.com.au/drumming-tutor/: <iframe src="/drumming-tutor/" style="width:100%;height:900px;border:0;" title="LRRecords Drummer"></iframe>
  • If using a different published URL, replace src with that URL.

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.

1 participant