Remove the mirror repo URL from edit form since it can't be changed

[lng2020]

Before:

After:

Or just change the background color of the input box 🤔 ?

[techknowlogick]

I believe that is there so that the user is aware of which URL they are modifying since push mirrors can have multiple URLs.

[lunny]

At least, it could be a readonly input box.

[lng2020]

I believe that is there so that the user is aware of which URL they are modifying since push mirrors can have multiple URLs.

There are different lines in the table so different records are just not easy to mix up. I deleted the input box just because the edit form title is Edit mirror sync internal.

At least, it could be a readonly input box.

The input box is read-only but will be active if the mouse clicks, which is easily confused.

PS:

I think the table also should add one column to indicate the current interval time.

So let's vote 😸

1. remove the URL input box or disable the input box to non-clickable. 😄 for first, 🚀 for second.
2. add a column to indicate the current sync interval time. 👍 or 👎

[puni9869]

I believe that is there so that the user is aware of which URL they are modifying since push mirrors can have multiple URLs.

There are different lines in the table so different records are just not easy to mix up. I deleted the input box just because the edit form title is Edit mirror sync internal.

At least, it could be a readonly input box.

The input box is read-only but will be active if the mouse clicks, which is easily confused.

PS: I think the table also should add one column to indicate the current interval time.

So let's vote 😸

1. remove the URL input box or disable the input box to non-clickable. 😄 for first, 🚀 for second.
2. add a column to indicate the current sync interval time. 👍 or 👎

I like this initiative

I have a question for you
1 How will you show the user that for which url you are modifiying if you are using the modal
2. I agree with the new design flow 👍 🚀 but I want to see the mock first. It will hamper the mobile view, if you add a new column in the current code, you need to modify that segment of view for mobile.

[puni9869]

At least, it could be a readonly input box.

it is readonly and disabled

[puni9869]

Here was approach
#26151

PS: please see the screenshots in the PR desc.

[lng2020]

At least, it could be a readonly input box.

it is readonly and disabled

No. It's read only but not disabled. see (https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/readonly#attribute_interactions)

[lng2020]

I believe that is there so that the user is aware of which URL they are modifying since push mirrors can have multiple URLs.

There are different lines in the table so different records are just not easy to mix up. I deleted the input box just because the edit form title is Edit mirror sync internal.

At least, it could be a readonly input box.

The input box is read-only but will be active if the mouse clicks, which is easily confused.
PS: I think the table also should add one column to indicate the current interval time.
So let's vote 😸

1. remove the URL input box or disable the input box to non-clickable. 😄 for first, 🚀 for second.
2. add a column to indicate the current sync interval time. 👍 or 👎

I like this initiative

I have a question for you 1 How will you show the user that for which url you are modifiying if you are using the modal 2. I agree with the new design flow 👍 🚀 but I want to see the mock first. It will hamper the mobile view, if you add a new column in the current code, you need to modify that segment of view for mobile.

That's true. I wonder if the row has the space for another column. And the current column will display unnormal if I change the browser width too.

[puni9869]

That's true. I wonder if the row has the space for another column. And the current column will display unnormal if I change the browser width too.

Think about it, there is always a better solution, we can discuss on the designs. or can take help/feedbacks from other maintainers