Skip to content

feat: separate CLI Tutor Mode helptext#1700

Merged
lidel merged 8 commits intomasterfrom
feat/separate-tutormode-helptext
Jan 5, 2021
Merged

feat: separate CLI Tutor Mode helptext#1700
lidel merged 8 commits intomasterfrom
feat/separate-tutormode-helptext

Conversation

@jessicaschilling
Copy link
Contributor

@jessicaschilling jessicaschilling commented Dec 17, 2020

Closes #1686.

This PR separates out the i18n key cliModal.extraNotes so we can display different CLI Tutor Mode modal helptext in different situations:

  • "IPFS Config" modal (already existed)
  • "Set Pinning" command in Files screen context menu (new in this PR)

To do:
The i18n keys are split out, but I'm missing something that's probably obvious in terms of getting helptext to display when a tutor-mode modal is invoked from the context menu. @lidel or @rafaelramalho19, do you mind a quick look if you get the chance? 🙏

@jessicaschilling
Copy link
Contributor Author

Thanks, @rafaelramalho19, but can you please look at the "to do" item when you get a chance? Thank you!

@rafaelramalho19
Copy link
Contributor

@jessicaschilling by help text you mean the title html attribute?

@jessicaschilling
Copy link
Contributor Author

@rafaelramalho19 - cliModal.ExtraNotes is customized as part of this work (that's the helptext), but I can't get it to display conditionally for the "Set pinning" command under the "three dots" context menu.

@jessicaschilling
Copy link
Contributor Author

Thanks, @rafaelramalho19!

@lidel - can you please have a look when you get the chance? Would be one more item done in the pinning epic ...

Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

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

LGTM but we should make key names more explicit,
will save us time when debugging bugs in the future :)

@jessicaschilling mind updating places where below keys are used?

jessicaschilling and others added 3 commits January 5, 2021 08:30
Co-authored-by: Marcin Rataj <lidel@lidel.org>
Co-authored-by: Marcin Rataj <lidel@lidel.org>
@jessicaschilling jessicaschilling requested a review from lidel January 5, 2021 15:34
@jessicaschilling
Copy link
Contributor Author

@lidel - good call. Updated. Please merge at your leisure ...

@lidel lidel merged commit 91e2f36 into master Jan 5, 2021
@lidel lidel deleted the feat/separate-tutormode-helptext branch January 5, 2021 16:23
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.

Pinning services: amend CLI tutor mode helptext

3 participants