Skip to content

Commit

Permalink
Update progress, add some knowledge
Browse files Browse the repository at this point in the history
  • Loading branch information
kaixin-hc committed Feb 25, 2024
1 parent 0116efe commit 6bdcfe9
Show file tree
Hide file tree
Showing 3 changed files with 25 additions and 4 deletions.
10 changes: 10 additions & 0 deletions students/kaixin-hc/knowledge.md
Original file line number Diff line number Diff line change
Expand Up @@ -106,3 +106,13 @@ Just needed to note this, and consider other options. Refactoring my code allowe
[source](https://dev.to/lofiandcode/
can-map-mutate-the-original-array-yes-dmb)
</panel>

### CSS Vendor Locking:

In the past, different browsers used different prefixes for CSS properties, so developers would often extensively cover the available cases with prefixes.

[Stylelint removes vendor prefixes](https://stackoverflow.com/questions/76144712/visual-studio-code-warning-also-define-the-standard-property-background-clip)

There is a danger for specific properties... eg [background-clip: text](https://stackoverflow.com/questions/75688924/background-clip-text-isnt-working-at-all-in-css) ... but for other cases in the markbind code, like `background-clip: padding-box`, it is no longer to prefix in order to obtain the desired behaviour.

Background reading on vendor prefixing: [Is vendor prefixing dead?](https://css-tricks.com/is-vendor-prefixing-dead/)
2 changes: 1 addition & 1 deletion students/kaixin-hc/observations.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
### Project: Foo
### Project: Godot

Give an intro to the project here ...

Expand Down
17 changes: 14 additions & 3 deletions students/kaixin-hc/progress.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,18 @@
```
| Week | Achievements |
| ---- | ------------ |
| 1 | Prepared idea & helped facillitate Saturday code sprint - MarkBind website making |
| 2 | Reviewed and merged PR: [Annotate support different labels #2388](https://github.com/MarkBind/markbind/pull/2388) |
| 2 | Reviewed and merged PR: [Annotate support different labels #2388](https://github.com/MarkBind/markbind/pull/2388) - and [added contributor](https://github.com/MarkBind/markbind/pull/2390) |
| 2 | Repo cleaning - investigated, reviewed available information, sometimes closed issues: [PageNav cannot handle duplicate headings #633](https://github.com/MarkBind/markbind/issues/633), [ PUML not working on CI deployment (Teammates) due to lack of graphviz #1903 ](https://github.com/MarkBind/markbind/issues/1903), [ Combine the syntax cheat sheet and full syntax reference pages #2091 ](https://github.com/MarkBind/markbind/issues/2091) |
```
| 3 | Product Roadmap. Investigated methods to plot out product roadmap, settling on GitHub Projects. Set up the project and categories with the team and sorted issues into the roadmap. ([Roadmap here](https://github.com/orgs/MarkBind/projects/1)) - [Related issue](https://github.com/MarkBind/markbind/issues/813) |
| 3-6 | Repo cleaning - Add tags to issues, add items to project road map, add summaries to issues. Examples : [Combine the syntax cheat sheet and full syntax reference pages #2091](https://github.com/MarkBind/markbind/issues/2091#issuecomment-1909719113), [Pop-Ups like tooltip/trigger do not work within the tree component](https://github.com/MarkBind/markbind/issues/1831), [ Add horizontal spacing between the icon and text in the custom icon list #2352 ](https://github.com/MarkBind/markbind/issues/2352) |
| 3 | My PR was merged: [Chore: Remove unused CSS for seamless panel styling #2407](https://github.com/MarkBind/markbind/pull/2407) |
| 4 | Merged and reviewed PR: [ Enhance search performance of algolia plugin #2406 ](https://github.com/MarkBind/markbind/pull/2406) |
| 4 | Reviewed and discussed PR (eventually closed): [Automatically add line-numbers when soft-wrapping (#2404) #2413 ](https://github.com/MarkBind/markbind/pull/2413) |
| 5 | Investigate [.gitignore not generated when init #2416](https://github.com/MarkBind/markbind/issues/2416) |
| 5 | Investigate [Tooltips seem to add an extra space in front #2419 ](https://github.com/MarkBind/markbind/issues/2419) |
| 5 | Reviewed PR: [MarkBind Template for Software Project Documentation #2400](https://github.com/MarkBind/markbind/pull/2400) |
| 5 | Add contributors in PRs with all-contributor: [yiwen101 1](https://github.com/MarkBind/markbind/pull/2437)[2](https://github.com/MarkBind/markbind/pull/2435), [luminousleek](https://github.com/MarkBind/markbind/pull/2428), [KevinEyo1](https://github.com/MarkBind/markbind/pull/2422)
| 6 | Pair programming / in person discussion with 3281 students |
| 6 | Reviewed PR: [Migrate stylelint to latest version #2292 #2425](https://github.com/MarkBind/markbind/pull/2425) |
| 6 | Merged and reviewed PR: [ Utilize GitHub Actions to aid checking of commit message #2429 ](https://github.com/MarkBind/markbind/pull/2429) |
| 6 | Merged and reviewed PR: [ Fix external styles and script not hoisted #2414 ](https://github.com/MarkBind/markbind/pull/2414) |

0 comments on commit 6bdcfe9

Please sign in to comment.