diff options
| author | silverwind <me@silverwind.io> | 2023-09-19 00:05:31 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-09-18 22:05:31 +0000 |
| commit | 8099238618f6573f1eb5cfeeb0902b641e7121ab (patch) | |
| tree | 34b6425a3838f52d56c052f10eb2b2328c680c37 /tests/e2e | |
| parent | e644cc9448a9271bba9789507dcef0984ac02d7f (diff) | |
| download | gitea-8099238618f6573f1eb5cfeeb0902b641e7121ab.tar.gz gitea-8099238618f6573f1eb5cfeeb0902b641e7121ab.zip | |
Change green buttons to primary color (#27099)
I think it's better if the primary actions have primary color instead of
green which fits better into the overall single-color UI design. This PR
currently replaces every green button with primary:
<img width="141" alt="Screenshot 2023-09-16 at 14 07 59"
src="https://github.com/go-gitea/gitea/assets/115237/843c1e50-4fb2-4ec6-84ba-0efb9472dcbe">
<img width="161" alt="Screenshot 2023-09-16 at 14 07 51"
src="https://github.com/go-gitea/gitea/assets/115237/9442195a-a3b2-4a42-b262-8377d6f5c0d1">
Modal actions now use uncolored/primary instead of previous green/red
colors. I also removed the box-shadow on all basic buttons:
<img width="259" alt="Screenshot 2023-09-16 at 14 16 39"
src="https://github.com/go-gitea/gitea/assets/115237/5beea529-127a-44b0-8d4c-afa7b034a490">
<img width="261" alt="Screenshot 2023-09-16 at 14 17 42"
src="https://github.com/go-gitea/gitea/assets/115237/4757f7b2-4d46-49bc-a797-38bb28437b88">
The change currently includes the "Merge PR" button, for which we might
want to make an exception to match the icon color there:
<img width="442" alt="Screenshot 2023-09-16 at 14 33 53"
src="https://github.com/go-gitea/gitea/assets/115237/993ac1a5-c94d-4895-b76c-0d872181a70b">
Diffstat (limited to 'tests/e2e')
| -rw-r--r-- | tests/e2e/example.test.e2e.js | 4 | ||||
| -rw-r--r-- | tests/e2e/utils_e2e.js | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/tests/e2e/example.test.e2e.js b/tests/e2e/example.test.e2e.js index 870c28cc77..c741663a38 100644 --- a/tests/e2e/example.test.e2e.js +++ b/tests/e2e/example.test.e2e.js @@ -20,7 +20,7 @@ test('Test Register Form', async ({page}, workerInfo) => { await page.type('input[name=email]', `e2e-test-${workerInfo.workerIndex}@test.com`); await page.type('input[name=password]', 'test123test123'); await page.type('input[name=retype]', 'test123test123'); - await page.click('form button.ui.green.button:visible'); + await page.click('form button.ui.primary.button:visible'); // Make sure we routed to the home page. Else login failed. await expect(page.url()).toBe(`${workerInfo.project.use.baseURL}/`); await expect(page.locator('.dashboard-navbar span>img.ui.avatar')).toBeVisible(); @@ -35,7 +35,7 @@ test('Test Login Form', async ({page}, workerInfo) => { await page.type('input[name=user_name]', `user2`); await page.type('input[name=password]', `password`); - await page.click('form button.ui.green.button:visible'); + await page.click('form button.ui.primary.button:visible'); await page.waitForLoadState('networkidle'); diff --git a/tests/e2e/utils_e2e.js b/tests/e2e/utils_e2e.js index ca6bde8db7..fba13ab426 100644 --- a/tests/e2e/utils_e2e.js +++ b/tests/e2e/utils_e2e.js @@ -18,7 +18,7 @@ export async function login_user(browser, workerInfo, user) { // Fill out form await page.type('input[name=user_name]', user); await page.type('input[name=password]', LOGIN_PASSWORD); - await page.click('form button.ui.green.button:visible'); + await page.click('form button.ui.primary.button:visible'); await page.waitForLoadState('networkidle'); |