Skip to content

site: add collections rules#429

Merged
kpsherva merged 2 commits intoinveniosoftware:masterfrom
jrcastro2:add-collections-style
Mar 25, 2026
Merged

site: add collections rules#429
kpsherva merged 2 commits intoinveniosoftware:masterfrom
jrcastro2:add-collections-style

Conversation

@jrcastro2
Copy link
Copy Markdown
Contributor

@jrcastro2 jrcastro2 force-pushed the add-collections-style branch from 959b7d7 to 91ed965 Compare January 21, 2026 09:28
@jrcastro2 jrcastro2 force-pushed the add-collections-style branch from 4c18b4d to 4c68c3a Compare January 28, 2026 14:23
Comment on lines +720 to +744
/*-------------------------
<<<<<<< Updated upstream
Collections Table Styling
=======
Job Logs Styling
-------------------------*/

/* Log table container */
.job-log-table {
max-height: 80vh;
overflow-y: auto;

/* Subtask container - subtle indentation for subtask logs without border */
.subtask-container {
padding: 0.5em;
margin: 0.5em 0 0.5em 1em;
background-color: #fafafa;
}
}

/*-------------------------
Collections Styling
>>>>>>> Stashed changes
(Used by invenio-communities)
-------------------------*/
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leak from other PR

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, thx, also removed some more code that is not needed, thx for this.

@ptamarit
Copy link
Copy Markdown
Member

I'm wondering if we could rely more on Semantic UI and write a bit less CSS?

@jrcastro2
Copy link
Copy Markdown
Contributor Author

I'm wondering if we could rely more on Semantic UI and write a bit less CSS?

Most of the CSS is for the drag and drop, and to reproduce the browse style page, in general I tried to stick to SUI as much as I could. Happy to hear some improvements or alternatives :)

@jrcastro2 jrcastro2 force-pushed the add-collections-style branch from 4c68c3a to a6d3d10 Compare February 23, 2026 13:13
ptamarit
ptamarit approved these changes Mar 2, 2026
Comment thread invenio_theme/assets/semantic-ui/less/invenio_theme/theme/globals/site.overrides Outdated
@jrcastro2 jrcastro2 force-pushed the add-collections-style branch from a6d3d10 to 2f4cfea Compare March 25, 2026 12:07
@kpsherva kpsherva merged commit db0197f into inveniosoftware:master Mar 25, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 - Implement full collections management + Browse tab visibility rules

4 participants