-
Notifications
You must be signed in to change notification settings - Fork 128
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
WET-484 Rotating features for Home and News #2424
base: master
Are you sure you want to change the base?
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.
My review is incomplete. Please do all the cleanup/update and use only supported component/pattern.
</div> | ||
</form> | ||
</details> | ||
<section class="lnkbx well"> |
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.
lnkbx
?
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.
@duboisp I've deprecated the version 2 of the working example as it was implemented. The "lnkbx" class add a margin-bottom of 10 pixels to every list item of the "Resources" section. I don't see any value of having this class.
<div class="mrgn-bttm-md" data-wb-randomize='{ "shuffle": true, "selector": "a", "toggle": "hide" }'> | ||
<a href="#" class="hide"> | ||
<figure> | ||
<img class="img-responsive full-width" src="https://via.placeholder.com/1170x347/F5F5F5/000000.png" alt=""> | ||
<figcaption class="well brdr-rds-0"> | ||
<span class="h4 mrgn-tp-0 mrgn-bttm-0 mrgn-lft-0">Image caption</span> | ||
</figcaption> | ||
</figure> | ||
</a> |
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.
That is a deprecated pattern when compared to our contextual feature pattern.
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.
@duboisp When deprecating a version do we need to modify the content or we should deprecated as is?
</figure> | ||
</a> | ||
</div> | ||
<div class="clearfix"> |
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.
clearfix
? That would not be needed when using the bootstrap grid
<div class="col-md-9"> | ||
<table class="wb-tables table nws-tbl" id="dataset-filter" aria-live="polite" data-wb-tables='{ |
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.
Missing something here?
No description provided.