added padding to the citation#646
Conversation
WalkthroughTwo Explorer components are updated to improve the visual layout and text consistency of schematic attribution. The attribution text is now wrapped in a padded grid row/column structure, and both components prepend "Schematics courtesy of" to the attribution source text. ChangesSchematic Attribution Layout and Text Updates
Possibly Related PRs
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
src/components/Explorer/TabSection.js (1)
96-100: 🛠️ Refactor suggestion | 🟠 MajorDuplicated attribution block.
This attribution block is identical to the one in
AccordionTabSection.js(lines 132-136). Please see the refactoring suggestion in that file to extract this into a shared component.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: a237b1d0-4110-4c12-90aa-738e29853f4c
📒 Files selected for processing (2)
src/components/Explorer/AccordionTabSection.jssrc/components/Explorer/TabSection.js
| <Row style={{padding: "1rem"}}> | ||
| <Col> | ||
| <small>Schematics courtesy of Bajema, Rachel, Supriya Bidanta, Ellen M. Quardokus, Bruce W. Herr II, and Katy Börner. December 2025. https://humanatlas.io/2d-ftu-illustrations, Accessed on June 1, 2026.</small> | ||
| </Col> | ||
| </Row> |
There was a problem hiding this comment.
🛠️ Refactor suggestion | 🟠 Major | ⚡ Quick win
Extract duplicated attribution block into a shared component.
The attribution layout and text are identical in both AccordionTabSection.js (lines 132-136) and TabSection.js (lines 96-100). This violates the DRY principle and creates a maintenance burden—any future changes to the citation or layout will require updates in multiple locations.
♻️ Suggested refactor: create a shared component
Create a new shared component SchematicAttribution.js:
import React from 'react';
import { Row, Col } from 'reactstrap';
const SchematicAttribution = () => {
return (
<Row style={{ padding: "1rem" }}>
<Col>
<small>
Schematics courtesy of Bajema, Rachel, Supriya Bidanta, Ellen M. Quardokus,
Bruce W. Herr II, and Katy Börner. December 2025.
https://humanatlas.io/2d-ftu-illustrations, Accessed on June 1, 2026.
</small>
</Col>
</Row>
);
};
export default SchematicAttribution;Then import and use it in both files:
+import SchematicAttribution from './SchematicAttribution';- <Row style={{padding: "1rem"}}>
- <Col>
- <small>Schematics courtesy of Bajema, Rachel, Supriya Bidanta, Ellen M. Quardokus, Bruce W. Herr II, and Katy Börner. December 2025. https://humanatlas.io/2d-ftu-illustrations, Accessed on June 1, 2026.</small>
- </Col>
- </Row>
+ <SchematicAttribution />📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <Row style={{padding: "1rem"}}> | |
| <Col> | |
| <small>Schematics courtesy of Bajema, Rachel, Supriya Bidanta, Ellen M. Quardokus, Bruce W. Herr II, and Katy Börner. December 2025. https://humanatlas.io/2d-ftu-illustrations, Accessed on June 1, 2026.</small> | |
| </Col> | |
| </Row> | |
| <SchematicAttribution /> |
Summary by CodeRabbit