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

feat(lh): add lighthouse mobile report #25

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

loickaczmarek
Copy link
Contributor

@loickaczmarek loickaczmarek commented Feb 12, 2024

In french ( english below )

Hello,

Je propose d'ajouter une analyse lighthouse mobile, avec une taille d'écran de 430 par 932, qui correspond à la résolution d'un iphone 14 depuis le simulateur mobile chrome.

Pour ça, j'ai ajouté un élément "mobile" dans le rapport lighthouse, pour ne pas casser le fonctionnement actuel.
La sortie du rapport contient un élément mobile, à la fois pour les rapports mais aussi pour la dernière analyse.

J'ai aussi modernisé une partie du code, en ajoutant quelques tests avec Jest, et en refactorant la partie qui présente les données lighthouse pour la sortie d'api.

Je n'ai pas ajouté l'affichage dans le plugin Sonar, car il pourrait être intéressant de procéder de la même manière pour le rapport desktop existant.

exemple :

{
    deployments: {
        ...
        lighthouse: [{
            desktop: {...},
            mobile: {...}
        }]
    },
    lastAnalysis: {
        ...
        lighthouse: {
            desktop: {...},
            mobile: {...}
        }
    }
}

D'ailleurs, sur le même modèle, je pense qu'ajouter une analyse ecoindex mobile est aussi possible.


English :

Hello,

I propose to add a mobile lighthouse analysis, with a screen size of 430 by 932, which corresponds to the resolution of an iphone 14 from the chrome mobile simulator.

For this, I've added a "mobile" element to the lighthouse report, so as not to break the current operation.
The report output contains a mobile element, both for reports and for the latest analysis.

I've also modernized some of the code, adding some tests with Jest, and refactoring the part that presents the lighthouse data for the api output.

I haven't added the display in the Sonar plugin, as it might be interesting to do the same for the existing desktop report.

example :

{
    deployments: {
        ...
        lighthouse: [{
            desktop: {...},
            mobile: {...}
        }]
    },
    lastAnalysis: {
        ...
        lighthouse: {
            desktop: {...},
            mobile: {...}
        }
    }
}

By the way, on the same model, I think adding a mobile ecoindex analysis is also possible.

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.

1 participant