Skip to content

Fix-of-issue-653 #698

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

Aditya-138-12
Copy link

@Aditya-138-12 Aditya-138-12 commented Jun 22, 2024

Changes / Fixes

  1. Added a shortcode file for tab.html.
  2. Modified tabs.html file for href, scrolling and removed prefix "nav-" from the id of tag of tablist, Added js Script in tabs.html for making the url change in adress bar when a tab is pressed.
  3. Modified index.md, added tabID in {{< tab tabName="Investment" tabID="Investment">}} for all tabs.

This PR fixes #653

Copy link

netlify bot commented Jun 22, 2024

Deploy Preview for tag-app-delivery ready!

Name Link
🔨 Latest commit 07f909d
🔍 Latest deploy log https://app.netlify.com/sites/tag-app-delivery/deploys/6676d7ab4feae800080c1cd4
😎 Deploy Preview https://deploy-preview-698--tag-app-delivery.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

…, scrolling and removed prefix nav- from the id of <a> tag of tablist

Signed-off-by: ADITYA SAROHA <[email protected]>
@Aditya-138-12 Aditya-138-12 deleted the issue-653-fix branch June 22, 2024 14:07
@Aditya-138-12 Aditya-138-12 restored the issue-653-fix branch February 27, 2025 22:04
@Aditya-138-12 Aditya-138-12 reopened this Feb 27, 2025
@github-actions github-actions bot added the website Related to TAG website. label Feb 27, 2025
@Aditya-138-12
Copy link
Author

Aditya-138-12 commented Feb 27, 2025

Hi, @abangser
I completely forgot about this issue taken by me, I am so soory for that...
Could you please take a look?

@abangser
Copy link
Collaborator

abangser commented Mar 1, 2025

No stress @Aditya-138-12 , thanks for taking the time to do this! I just had a look at the preview branch and I am not sure it quite behaves as I would have expected. Specifically I would expect:

  1. When I click on the header of an aspect within the table
    a. Expected: the URL should display the aspect I have selected
    b. Actual: The URL gets a # but no other indicator, thereby not allowing users to link to a specific aspect by URL
  2. When I click on the header of an aspect within the table
    a. Expected: The page scrolls to the details section
    b. Actual: the details section (correctly) switches tabs, but the page does not scroll
  3. When I click on the 'anchor link' for a characteristics section in the details
    a. Expected: it should be unique and something I can share a direct URL to
    b. Actual: it is just #characteristics in the URL thereby not being unique to that aspect/level combo

These anchor tags prove to be quite tricky but I am not sure this specific change hits the mark just yet! Happy to keep reviewing as you make progress though 😊

@Aditya-138-12
Copy link
Author

Hi @abangser
Yeah, It looks quite tricky though, but I will try to fix them,
Thanks for your time!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
website Related to TAG website.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Platform Engineering Maturity Model tab improvement
2 participants