unfoldingWord / gateway-edit

Book Package harmonized view.
https://gatewayedit.com
MIT License
1 stars 4 forks source link

IDs for Floating Action Button items #416

Closed birchamp closed 9 months ago

birchamp commented 1 year ago

Work with Daniel on finding missing IDs.

danielklapp commented 1 year ago

Restoring minimized cards

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

PhotoNomad0 commented 1 year ago

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

jincypjose commented 1 year ago

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 commented 1 year ago

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

danielklapp commented 1 year ago

@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 commented 1 year ago

@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 commented 1 year ago

@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:

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

PhotoNomad0 commented 1 year ago

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 commented 1 year ago

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,

birchamp commented 1 year ago

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

jincypjose commented 1 year ago

@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.

danielklapp commented 11 months ago

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

Joel-C-Johnson commented 11 months ago

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/

  1. 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

  1. Minimize cards using the '-" icon in the cards
  2. 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 commented 11 months ago

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

elsylambert commented 11 months ago

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

danielklapp commented 11 months ago

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