Skip to content
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 1499 ui pattern demo - fp 1726 publications list #37

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Jul 26, 2022

Status

Better solution expected via FP-1770.

Overview

Create publication listing pattern and related patterns.

To Do

  • updates after sharing H.P. feedback with M.S.
    • do not highlight title
    • provide a manual design, because this is data-driven
  • consider splitting publications styles into:
    • "data-driven"
    • "manual-input"

Related

Changes

  • feat: .c-publication-preview
  • feat: .c-tag & .c-tag-list
  • feat: layout partial template for re-usable "how not to use" section of examples for patterns

Testing

Core Styles

  1. npm ci
  2. npm run build:css
  3. npm run start

Core CMS

  1. npm ci
  2. npm run build
  3. npm run start:css-demo

UI

Local
core-styles core-cms
core-styles c-pub-preview core-cms c-pub-preview
core-styles c-tag multiple core-cms c-tag multiple
core-styles c-tag-list ul core-cms c-tag-list ul
Remote

(no deploy process available)

No new solution for cross-repo testing this package (sans npm publish).

I hve an idea, but for now I'll just commit the dist temporarily.
Last commit I accidentally added outdated files into temp. dist folder.
Let first pub preview have border, even if it is not the first element.
- limit citation styles to first paragraph
- limit status styles to last paragraph (that is not a citation)
- fix status styles (there were just small things wrong)
- set padding for a paragraph that is neither citation nor status
@wesleyboar wesleyboar force-pushed the task/fp-1499-cms-pattern-library--fp-1726-publications-list branch from 240a076 to 3299c60 Compare July 27, 2022 00:02
@wesleyboar wesleyboar force-pushed the task/fp-1499-cms-pattern-library--fp-1726-publications-list branch from 1060da1 to 6d945d3 Compare July 28, 2022 21:12
@wesleyboar
Copy link
Member Author

Retiring this repo in favor of https://github.com/TACC/tup-ui.

@wesleyboar wesleyboar closed this Jul 28, 2022
@wesleyboar
Copy link
Member Author

wesleyboar commented Jul 29, 2022

The core-styles directory on https://github.com/TACC/tup-ui is frozen for now. Using this repo instead.

@wesleyboar wesleyboar reopened this Jul 29, 2022
@wesleyboar wesleyboar changed the base branch from main to task/fp-1499-cms-pattern-library July 29, 2022 20:30
@wesleyboar wesleyboar marked this pull request as ready for review August 8, 2022 20:23
@wesleyboar wesleyboar changed the title Task/fp 1499 cms pattern library fp 1726 publications list Task/fp 1499 ui pattern demo - fp 1726 publications list Aug 8, 2022
@wesleyboar
Copy link
Member Author

The styles and may change after designers share feedback.

Base automatically changed from task/fp-1499-cms-pattern-library to main August 16, 2022 17:34
@wesleyboar wesleyboar added the paused Started but not actively in progress label Aug 16, 2022
@wesleyboar wesleyboar closed this Oct 12, 2022
@wesleyboar
Copy link
Member Author

Revisit better solution via FP-1770.

@wesleyboar wesleyboar deleted the task/fp-1499-cms-pattern-library--fp-1726-publications-list branch November 30, 2022 05:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
paused Started but not actively in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant