Skip to content

Remove Scripture nav controls#75

Draft
imnasnainaec wants to merge 4 commits into
mainfrom
nix-scripture-controls
Draft

Remove Scripture nav controls#75
imnasnainaec wants to merge 4 commits into
mainfrom
nix-scripture-controls

Conversation

@imnasnainaec
Copy link
Copy Markdown
Contributor

@imnasnainaec imnasnainaec commented May 15, 2026

Resolves #68

One challenge with this has been posted for discussion in the Platform.Bible discord: https://discord.com/channels/1064938364597436416/1504844969566011463

Update: the Scripture nav control was only removed from the Scripture tab for "simple" mode, but is still there for "power" mode:
Screenshot 2026-05-15 135012

Screenshot 2026-05-15 135152 Screenshot 2026-05-15 135135

This change is Reviewable

@imnasnainaec imnasnainaec self-assigned this May 15, 2026
@imnasnainaec imnasnainaec added the 🟨Medium Medium-priority PR label May 15, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 15, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 16641349-c4d1-4bf7-955b-bb10bc7cf4ed

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch nix-scripture-controls

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@imnasnainaec imnasnainaec marked this pull request as draft May 15, 2026 13:27
imnasnainaec and others added 3 commits May 15, 2026 09:31
This reverts commits 81b1a89 and 2e30e30.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Use `useSetting('platform.interfaceMode', 'simple')` in InterlinearizerLoader
and render ScriptureNavControls only when the value is 'power'.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@imnasnainaec imnasnainaec force-pushed the nix-scripture-controls branch from c7606e9 to 062863b Compare May 15, 2026 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🟨Medium Medium-priority PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove Scripture Controls

1 participant