Skip to content

Commit

Permalink
Merge pull request #42 from yqni13/Development
Browse files Browse the repository at this point in the history
Development_v2.6.5
  • Loading branch information
yqni13 authored Jul 14, 2024
2 parents 038afca + 9537641 commit de587fb
Show file tree
Hide file tree
Showing 48 changed files with 3,729 additions and 2,610 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,13 +33,13 @@ jobs:

- name: Configure Git
run: |
git config --global user.email "yqni13@protonmail.com"
git config --global user.name "yqni13"
git config --global user.email "${{ github.actor }}@users.noreply.github.com"
git config --global user.name "${{ github.actor }}"
git remote set-url origin https://${{ secrets.PORTFOLIO_PAT }}@github.com/${{ github.repository }}.git
- name: Deploy to GitHub Pages
env:
GH_PAT: ${{ secrets.PORTFOLIO_PAT }}
run: |
git remote set-url origin https://${{ secrets.PORTFOLIO_PAT }}@github.com/yqni13/portfolio.git
npx angular-cli-ghpages --dir=dist/portfolio/browser --repo=https://${{ secrets.PORTFOLIO_PAT }}@github.com/yqni13/portfolio.git --no-silent
npx angular-cli-ghpages --dir=dist/portfolio/browser --repo=https://${{ secrets.PORTFOLIO_PAT }}@github.com/${{ github.repository }}.git --no-silent
24 changes: 13 additions & 11 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,6 @@ $\texttt{\color{teal}{v2.5.5}}$
</div>
<br>

This project was generated with [Angular CLI](https://github.com/angular/angular-cli) version 17.3.6.

### <a href="https://yqni13.github.io/portfolio/home">TRY LIVE DEMO</a>

<br>
Expand All @@ -45,7 +43,7 @@ deploy >> `npx angular-cli-ghpages --dir=dist/<projectname>/browser --repo=https
## Features
<dl>
<dd>:iphone: Responsive design 480px > width < 1440px via flexbox and media queries</dd>
<dd>:book: Multi-Page Layout (main navigation and portfolio menu)</dd>
<dd>:page_facing_up: SPA (Single-Page-Application)</dd>
<dd>:new_moon_with_face:/:sun_with_face: Dark/Light mode</dd>
<dd>:art: Customized style</dd>
<dd>:mag: Customized filter search</dd>
Expand Down Expand Up @@ -104,7 +102,9 @@ My portfolio comes with two theme settings: $\textsf{\color{gray}{Dark mode}}$ &
<img src="src/assets/readme_img/logo_ico/firefox_logo50.ico"> | <img src="src/assets/readme_img/logo_ico/chrome_logo50.ico"> | <img src="src/assets/readme_img/logo_ico/opera_logo50.ico"> | <img src="src/assets/readme_img/logo_ico/edge_logo50.ico"> | <img src="src/assets/readme_img/logo_ico/duckduckgo_logo50.ico"> | <img src="src/assets/readme_img/logo_ico/brave_logo50.ico">
|:------:|:------:|:------:|:------:|:------:|:------:|
|Firefox | Chrome | Opera | Edge | DuckGo | Brave |
|Yes | Yes | Yes | Yes | Yes | Yes |
|Yes | Yes | Yes | Yes | Yes* | Yes |

* This browser has problems displaying certain animations.

</center>
<br>
Expand All @@ -129,22 +129,24 @@ Specified unit tests in xyz.component.spec.ts files or other tests are my last s
## Updates

[list of all updates](src/docs/update_protocol.md)
### $\textsf{last\ update\ 2.5.0\ >>\ {\color{pink}2.5.5}}$
### $\textsf{last\ update\ 2.5.5\ >>\ {\color{pink}2.6.5}}$

- $\textsf{\color{red}Patch:}$ Addressing webpage stability issues: Disabled error alert because of fixed hosting.
- $\textsf{\color{red}Bugfix:}$ Default unit tests have all necessary declarations and imports. [Before: Template was not recognised as existing component because unit tests were missing the regarding declarations and imports.]
- $\textsf{\color{red}Bugfix:}$ Hosting works again (refreshing does not lead to error 404) and CI/CD pipeline adapted. [Before: Necessary options in deployment command missing. Customized workflow was missing.]
- $\textsf{\color{red}Bugfix:}$ Navigating to other page in menu resets scroll to top position (refresh gets back to current scroll position). [Before: Scroll position was used for next page so that it didn't start at top.]
- $\textsf{\color{red}Bugfix:}$ The component 'portfolio' shows the correct number of cards with the regarding data. [Before: Due to change of ESLint errors, json data was fetched by import but loaded single objects additional to default collection of objects. Needed adaption in import command to load data only from default.]
- $\textsf{\color{green}Change:}$ Added page scroll indicators (white arrow animation, left bottom) for the components 'about', 'cv' and 'portfolio'.
- $\textsf{\color{red}Patch:}$ Refactored code in terms of Google Guidelines HTML/CSS.
- $\textsf{\color{red}Patch:}$ Refactored code in terms of Google Guidelines Typescript.
- $\textsf{\color{red}Patch:}$ Refactored code in terms of Angular Best Practices.
- $\textsf{\color{red}Patch:}$ Refactored code to reuse user-data from parent in child component and reduce redundancy.
- $\textsf{\color{red}Bugfix:}$ Navigating to base url redirects to 'home' component and activates nav-menu item accordingly. [Before: Redirecting to 'home' component didn't set 'Home' nav-menu item active.]

<br>

### Aimed objectives for next $\textsf{\color{green}minor}$ update:
<dl>
<dd>- error handling for images failing to load</dd>
<dd>- error handling for exceptional cases/processes</dd>
<dd>- customized error page</dd>
<dd>- css animations for extending/hiding elements</dd>
<dd>- enable filtering portfolio cards with multiple keywords</dd>
<dd>- unit tests for all components</dd>
</dl>
<br>

Expand Down
6 changes: 4 additions & 2 deletions angular.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
"build": {
"builder": "@angular-devkit/build-angular:application",
"options": {
"aot": false,
"outputPath": "dist/portfolio",
"index": "src/index.html",
"browser": "src/main.ts",
Expand Down Expand Up @@ -54,10 +55,11 @@
},
{
"type": "anyComponentStyle",
"maximumWarning": "150kb",
"maximumError": "150kb"
"maximumWarning": "2000kb",
"maximumError": "3000kb"
}
],
"aot": true,
"outputHashing": "all"
},
"development": {
Expand Down
4 changes: 2 additions & 2 deletions eslint.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,15 +18,15 @@ module.exports = tseslint.config(
"error",
{
type: "attribute",
prefix: "app",
prefix: ["app", "yqni13Var"],
style: "camelCase",
},
],
"@angular-eslint/component-selector": [
"error",
{
type: "element",
prefix: ["app", "ng-template"],
prefix: ["app", "yqni13-template"],
style: "kebab-case",
},
],
Expand Down
Loading

0 comments on commit de587fb

Please sign in to comment.