-
Notifications
You must be signed in to change notification settings - Fork 2
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
[Gallery] Gallery tab header content #69
Comments
fixed with #84 |
@tom-hightower are we looking at the same page ? Can you review https://main--firefly--adobecom.hlx.page/community and let me know what are we missing and I can fix those for you. Other thing I noticed, where are these strings coming from ?
I don't see them in the en-US.json dictionary ? Are these Hardcoded strings ? |
Oh sorry, I was looking at the https://main--firefly--adobecom.hlx.live/community version. What is the difference between this and the The version you linked is closer, but still doesn't match the existing live site. The main issues I see are the spacing, color theming, content (strings that the community gallery team use are here), and that the "View your submissions" should only appear for a logged in user. |
@tom-hightower Spacing and other things are easy fixes and can deal with them tomorrow morning. Localized content now in community git repo altogether makes things a bit weird as we were piggybacking on en-us and other localized json in spl-webclient repo as a single source of truth but now we have a new repo that has different set of localized json that too just for the community page wondering if we should create a manual spreadsheet with key value pairs of strings currently being used on homepage and community page rather than trying to reuse the localized jsons from both these repos. Opinions ? Suggestions ? @kailasnadh790 For the ‘view submission’ button, is there a helper function that we use to check if the user is logged in or not? |
@tom-hightower Putting this issue on hold until we have clarity if firefly team wants to take just the homepage or both homepage + community page on the new platform. Will pick this up when we have clear direction on that. |
Firefly:
![image](https://private-user-images.githubusercontent.com/39915804/339867602-74568ec3-7460-49a3-b184-993033a28c5a.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk0MzU5NzIsIm5iZiI6MTczOTQzNTY3MiwicGF0aCI6Ii8zOTkxNTgwNC8zMzk4Njc2MDItNzQ1NjhlYzMtNzQ2MC00OWEzLWIxODQtOTkzMDMzYTI4YzVhLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTMlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjEzVDA4MzQzMlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTdlNjNmNjNmNGU4YjZiZDgwMmUxNjAzYjEzMjQ1NWRiMTUzMDcxMzEyN2QyNDcyZDFmNDRjM2RhZmU3MzkyZmMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.wuHEHF6jrRNyWsUaUAV2B4_IVXd6qHaIMA-CE9B6_aY)
Franklin:
![image](https://private-user-images.githubusercontent.com/39915804/339867677-594a8c70-56be-464f-8db5-66914ca24374.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk0MzU5NzIsIm5iZiI6MTczOTQzNTY3MiwicGF0aCI6Ii8zOTkxNTgwNC8zMzk4Njc2NzctNTk0YThjNzAtNTZiZS00NjRmLThkYjUtNjY5MTRjYTI0Mzc0LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTMlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjEzVDA4MzQzMlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWRhNTdiNGIyMDlhZTVlMTQ1M2I1NmNjMGYxZjRjOGJmZDdiZTQ5NjBmMGVmNWEyMTNiYWRiMDhkOWFmNGRiMGYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.idh5h0k6NzzbK5MMfKUPrgBhzd-kU8yPTYpZhqCetQU)
The text was updated successfully, but these errors were encountered: