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

Add Project Teasers: Plain text instead of html field and limitation to a few sentences #621

Open
jstet opened this issue Dec 13, 2023 · 3 comments
Assignees

Comments

@jstet
Copy link
Member

jstet commented Dec 13, 2023

The field currently called "Summary" should be called "Teaser" instead and not be a html field. It's comparable to the teaser of blog articles. It should be only a few sentences and not contain html.

To change this however, we would need to write some code that copies the text without html tags from the old field to a new field.

@friep do you agree?

@pr130
Copy link
Contributor

pr130 commented Dec 18, 2023

I see the problem. However, we do have a few projects that will never get an extensive description (in the sense of "problem", "data", "approach", "impact") but have a summary that is a) longer and b) includes links to outputs etc.

in addition, i like having the summary as an extra field because it's the amount of text that could be used for a longer report etc. so it'd be good to have API access to it (and not include it in the description).

finally, right now, there is no real common approach on how to start the summary so that it makes sense to be used as a "teaser" (i.e. some summaries just start very general and do not "tease" the reader).

My suggestion:

  • keep summary field
  • add new teaser field that really is a teaser/fits the preview (i.e. one sentence) -> i'll go through the projects and generate those teasers using chatgpt ;)

@jstet
Copy link
Member Author

jstet commented Jan 11, 2024

Sounds good to me. I'll add the teaser field!

@jstet jstet changed the title Project Teasers: Plain text instead of html field and limitation to a few sentences Add Project Teasers: Plain text instead of html field and limitation to a few sentences Jan 11, 2024
@jstet
Copy link
Member Author

jstet commented Jan 11, 2024

@friep so once you have added content to this field for all projects, I will change the logic of project db so that it displays the teaser in the cards and at the top if the sub page. I will also work on #632 ; displaying the summary as the main text of sub pages when there is no description.

@jstet jstet assigned jstet and pr130 Jan 11, 2024
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

No branches or pull requests

2 participants