-
Notifications
You must be signed in to change notification settings - Fork 1
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
task/FP-1260 Rename Component ReadMore to ShowMore #376
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also rename "references in comments." — FP-1260
I see at least five:
- three in docblock at top of file
- two at
/* Read More / Read Less */
taccsite_cms/static/site_cms/css/src/_imports/components/c-read-more.css
Outdated
Show resolved
Hide resolved
taccsite_cms/static/site_cms/css/src/_imports/components/c-read-more.css
Outdated
Show resolved
Hide resolved
…d-more.css Co-authored-by: W. Bomar <62723358+tacc-wbomar@users.noreply.github.com>
…d-more.css Co-authored-by: W. Bomar <62723358+tacc-wbomar@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File content looks good.
Oh, the file itself needs to be renamed 😅. We both missed that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to go after rename the file 👍🏼
I thought about that too, but I guess it matter haha 😅 |
Sorry for the late merge. I had no obstacle; I accidentally neglected it. |
Overview:
Rename .c-read-more to .c-show-more.
Related Jira tickets:
Summary of Changes:
.c-read-more
to.c-show-more
Testing Steps:
.c-read-more
in codebase and check if it changed to.c-show-more
UI Photos:
Notes: