Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IDs for Floating Action Button items #416

Closed
birchamp opened this issue Apr 17, 2023 · 17 comments
Closed

IDs for Floating Action Button items #416

birchamp opened this issue Apr 17, 2023 · 17 comments

Comments

@birchamp
Copy link

birchamp commented Apr 17, 2023

No description provided.

@birchamp
Copy link
Author

Work with Daniel on finding missing IDs.

@danielklapp
Copy link

danielklapp commented Apr 17, 2023

Restoring minimized cards

  • Parent container (div class="jss28") to have good meaningful ID that makes it unique.
  • Child containers (div class= "jss29") to have CSS class names of what components they are if possible.

Total of eight child containers I'm testing to restore:

  • Literal Translation
  • Original Source
  • Simplified Translation
  • translationNotes
  • translationAcademy
  • translationWords List
  • translationWords Article
  • translationQuestions

@PhotoNomad0
Copy link
Contributor

Notes:
also each card unique id for minimize and save.
also merge buttons and alignment buttons.

@jincypjose
Copy link
Contributor

jincypjose commented Apr 18, 2023

These are the id's to click on
show_minimized_cards
close_minimized_cards
restore_button_resource_card_tq for Translation questions
restore_title_resource_card_tq for Translation questions
restore_title_scripture_card_Literal_Translation for Literal Translation
restore_button_scripture_card_Literal_Translation for Literal Translation
restore_title_scripture_card_Original_Source for Original Source
restore_button_scripture_card_Original_Source for Original Source
restore_title_scripture_card_Simplified_Translation for Simplified Translation
restore_button_scripture_card_Simplified_Translation for Simplified Translation
restore_title_resource_card_ta for Translation academy
restore_button_resource_card_ta for Translation academy
restore_title_resource_card_twl for Translation Word List
restore_button_resource_card_twl for Translation Word List
restore_title_resource_card_twa for Translation Words Article
restore_button_resource_card_twa for Translation Words Article

@jincypjose
Copy link
Contributor

@danielklapp @elsylambert the fixes are in https://release-v2-0-0--gateway-edit.netlify.app
version- v1.2.0 build b660927

@danielklapp
Copy link

@jincypjose IDs for buttons is looking Great!
I've noticed it would be helpful to add IDs to the alignment buttons that specific to Literal Translation card, Original Source card, and Simplified Translation card.

@PhotoNomad0
Copy link
Contributor

@jincypjose IDs for buttons is looking Great! I've noticed it would be helpful to add IDs to the alignment buttons that specific to Literal Translation card, Original Source card, and Simplified Translation card.

We need to talk about this, since the scripture cards are not fixed, they can be swapped or changed point to about any scripture repo.

@PhotoNomad0
Copy link
Contributor

PhotoNomad0 commented May 15, 2023

@jincypjose @danielklapp - since each card could be changed to point to different scriptures, perhaps it is best to make the ids for the alignment buttons consistant with the other buttons. For example instead of id="alignment_icon_TARGET_LITERAL", do id="alignment_icon_TARGET_LITERAL" id="alignment_icon_scripture_card_0".

There are several ids in ScriptureCard.tsx that need to be changed:

  • valid_icon_${resourceId}
  • invalid_alignment_icon_${resourceId}
  • alignment_icon_${resourceId}

where "${resourceId}" should be replaced with "scripture_card_${cardNum}"

@PhotoNomad0
Copy link
Contributor

PhotoNomad0 commented May 15, 2023

Also in the FAB's we have:

id="restore_title_scripture_card_Original_Source"
id="restore_button_scripture_card_Original_Source"

to fix the id's make changes in the WorkspaceContainer.js of GWE. change the id values for the scripture cards in const cards = [ to be scripture_card_${cardNum}

@PhotoNomad0
Copy link
Contributor

To fix the title, in single-scripture-rcl ScriptureCard change

        onMinimize={onMinimize ? () => onMinimize(id) : null}

to be

        onMinimize={onMinimize ? () => onMinimize(id, scriptureLabel) : null}

And in resource-workspace-rcl useMinimizedCardsState,

  • add a state variable to persist titles const [minimizedCardTitles, setMinimizedCardTitles] = useUserLocalStorage('minimizedCardITitles', {});
  • need to add a title as second parameter to const minimizeCard = (id) => {
  • and in the function add the title to minimizedCardTitles
  • and need to change minimizedCards declaration to be a useMemo and return items that match contained in minimizedCardIds and replacing the titles if contained in minimizedCardTitles

@birchamp
Copy link
Author

@jincypjose please hand this off to @Joel-C-Johnson and let him know what's left to be done

@jincypjose
Copy link
Contributor

@Joel-C-Johnson i have did what need have to be done in ScriptureCard.tsx as bruce metioned above. Now the WorkspaceContainer.js need to be fixed with cardNum.

PhotoNomad0 added a commit that referenced this issue Jun 13, 2023
* bump (#407)

* Update latest into v1.2.0 (#408)

* QA Fail 372 / Edit/Alignments not saved (#398)

* updated dependencies.

* connect merge status

* revert markdown-translatable to 2.0.3 for GWE compatibility.

* patch for lexicon popups

* updated versions for translation help rcl and resource workspace rcl … (#418)

* updated versions for translation help rcl and resource workspace rcl for issue 416

* updated the versions for translation-helps-rcl and resource-workspace-rcl

* Feature jincy 395 (#422)

* updated versions for translation help rcl and resource workspace rcl for issue 416

* we removed selection context

* removed selection context

* updated version for single scripture rcl

* updated dependencies.

* updated dependencies.

* updated dependencies.

* updated dependencies.

* updated dependencies.

* updated dependencies.

* updated dependencies.

* fix the refrence crash with translation words

* resolved conflicts

* we updated dependencies

---------

Co-authored-by: blm <[email protected]>

* Updated single-scripture-rcl for aligning reference ranges (#420)

* Updated single-scripture-rcl for aligning reference ranges

* Updated version of gateway-edit

* Maybe changing yarn lock will make it happy

* Update version of single-scripture-rcl

* yarn locks are the bane of my existence

* Updated version of single-scripture-rcl

* Feature 400/ book branch (#412)

* QA Fail 372 / Edit/Alignments not saved (#398)

* updated dependencies.

* connect merge status

* revert markdown-translatable to 2.0.3 for GWE compatibility.

* updated dependncies

* updated dependncies

* updated dependncies

* updated dependncies

* updated dependncies

* bump version

* updated dependencies.

* updated dependencies.

* updated dependencies.

* updated dependencies.

* fix timeouts

* fix timeouts

* updated dependncies

* updated dependencies.

* updated dependencies.

* move fetch of original language back into app.  consolidate state variables

* move fetch of original language back into app.  consolidate state variables

* updated dependencies.

* bump timeout

* bump timeout

* bump timeout

* updated dependencies. (#436)

* added debug logging

* added debug logging

* cleanup

* cleanup

* updated dependencies (#441)

* updated dependencies.

* updated dependencies.

* Fixed aligner null data on accept with no changes (#425)

* fixes for loading/saving repo.

* updated dependencies.

* fixes for saving resource.

* updated dependencies.

* updated dependencies.

* cleanup.

* updated latest word-aligner-rcl

* updated dependencies.

* cleanup.

* cleanup.

* added title for cards

* updated the RCL version

* updated the dependencies

---------

Co-authored-by: Bruce McLean <[email protected]>
Co-authored-by: jincypjose <[email protected]>
Co-authored-by: Kintsoogi <[email protected]>
@danielklapp
Copy link

@Joel-C-Johnson can you give an update about the changes you’ve made and how we can test those changes?

@Joel-C-Johnson
Copy link
Collaborator

Hi @danielklapp, I will right the steps here and if you have doubts please ping me on Discord.
1. Login into gatewayedit - https://gateway-edit.netlify.app/
2. Open inspect and click on each card using the "pick element tool"

image

3. You can able to see unique id's for cards and icons
4. Minimize cards using the '-" icon in the cards
5. you can able see recent cards on the right side

image

6. click on it and you can see the list of cards with unique id and names

image

@danielklapp
Copy link

@Joel-C-Johnson thanks, just wanted to make sure my testing covered all work you guys did.

@elsylambert
Copy link
Contributor

Looks good in v2.1.0 build 66dd56e QA. Seeing IDs in Restore card button icons.

@danielklapp
Copy link

Looks good to me. I see unique IDs for all the buttons listed. Tested with v2.1.0 build 66dd56e QA.

kintsoogi pushed a commit that referenced this issue Sep 27, 2023
removed selection context
move fetch of original language back into app.  consolidate state variables
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants