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

554 skille funksjonalitet kunne refreshe og advarsel #561

Merged

Conversation

malinnsnieske
Copy link
Collaborator

Background

Skille på funksjonalitet for å refreshe svar (skal være mulig hele tiden) og gi advarsel for at svaret må oppdateres (skal dukke opp når svaret er utgått)

Solution

La til funksjonaliteten om var i RefreshAnswer inn i LastUpdated for å tilpasse skrift og ikoner. Siden det er ikoner uten tekst, la jeg til Tooltip komponenter som gjør at tekst dukker opp når man hovrer over ikoner.
Fjernet RefreshAnswer, og også LastUpdatedQuestionPage-filen for å slippe å doble funksjonalitet.

Resolves #554

@malinnsnieske malinnsnieske linked an issue Dec 16, 2024 that may be closed by this pull request
@malinnsnieske malinnsnieske changed the base branch from main to staging December 16, 2024 14:34
@malinnsnieske malinnsnieske force-pushed the 554-skille-funksjonalitet-kunne-refreshe-og-advarsel branch from 78d92a1 to 711a9f2 Compare December 16, 2024 14:35
Copy link
Collaborator

@JegHeterGyda JegHeterGyda left a comment

Choose a reason for hiding this comment

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

Ser bra ut

@malinnsnieske malinnsnieske merged commit 94e2672 into staging Dec 18, 2024
@malinnsnieske malinnsnieske deleted the 554-skille-funksjonalitet-kunne-refreshe-og-advarsel branch December 18, 2024 12:50
starheim98 pushed a commit that referenced this pull request Dec 19, 2024
chribjel added a commit that referenced this pull request Jan 10, 2025
* fix linjeskift on comments in detailed view (#562)

Co-authored-by: Mailn Nifeli Snieske <[email protected]>

* separate functionality between refreshing answer and warning (#561)

Co-authored-by: Mailn Nifeli Snieske <[email protected]>

* reduced se-mer width

* added tooltip and removed text (#567)

* make tables endpoint public and create tableId endpoint (#573)

Co-authored-by: Mailn Nifeli Snieske <[email protected]>

* dont stretch page if not needed (#572)

* add get schemas endpoint, fix cors frisk, fix redirect for new metadata (#575)

* add get schemas endpoint, fix cors frisk, fix redirect for new metadata
---------

Co-authored-by: Mailn Nifeli Snieske <[email protected]>

* fix cors (#578)

Co-authored-by: Mailn Nifeli Snieske <[email protected]>

* remove frisk config (#579)

Co-authored-by: Mailn Nifeli Snieske <[email protected]>

---------

Co-authored-by: malinnsnieske <[email protected]>
Co-authored-by: Mailn Nifeli Snieske <[email protected]>
Co-authored-by: starheim98 <[email protected]>
Co-authored-by: Andreas <[email protected]>
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.

Skille funksjonalitet: kunne refreshe og advarsel
3 participants