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

Feature coding camp framework #3728

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Conversation

ylivuoto
Copy link
Contributor

What's new

First feature implementation for coding camp course template.

The feature

User profile component.

  • Participant could create a profile and add a picture in it
  • Enable providing a self description
  • Enable providing a link to own website or to video-intro etc.

Profile data is stored into personal folder, "profile" document, into document settings.

Comment on lines +464 to +466
ngOnInit() {
console.log("File select init.");
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is potentially unnnecessary.

Comment on lines -8 to +9
background: black;
opacity: 0.5;
background-color: #20202020;
backdrop-filter: blur(7px);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This change was just an quick try/upgrade. Not necessary if you don't like :)


export type InputType = "TEXTAREA" | "TEXT";

@Component({
Copy link
Contributor

Choose a reason for hiding this comment

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

Mietin että ei ehkä sittenkään ole kovin mielekästä pitää tätä erillisenä komponenttina. Ajatus oli että textfield-plugin ja tämä sekä tulevaisuudessa muut mahdolliset kentät voisi käyttää ns "yleiskäyttöistä" tekstiboksia, mutta nyt kun katsoo tarkemmin mitä tuo user-profile-component tuolla tekstilaatikoillaan tekee niin ehkä ylläpidon kannalta on selkeämpi vain jättää tämä pois. Sen sijaan profiilikomponentissa voisi suoraan käyttää ja <textarea>-elementtejä.

Tällä hetkellä ainoa toisto mistä pääsisi eroon erillisellä komponentilla on nuo varoitusvärit, mutta niiden css-pätkien kopsiminen toiseen paikkaan ei ole kovin iso homma vrt. uuden komponentin ylläpito. Nuo nykyiset komponentin bindaukset saa helposti suoraan siihen -elementtiin, eikä silloin tarvitse kikkailla angularin inputtien ja eventtien kanssa. Esim valueChange voi olla suoraan ngModelChange-bindaus ja sisällön saa suoraan kaksisuuntaisella ngModel-bindauksella

@ylivuoto ylivuoto force-pushed the feature-coding-camp-framework branch from d869257 to 4c8cc50 Compare November 1, 2024 07:26
@@ -461,6 +461,9 @@ export class FileSelectManagerComponent {
}[] = [];

constructor(public cdr: ChangeDetectorRef) {}
ngOnInit() {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tein pakotetun puskun tässä välissä, Simon tekemät muutokset on temp haarassa tallessa.

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.

2 participants