[pve-devel] applied: [PATCH manager v2] fix #5106: ui: ceph status: make column wrap the summary text on expand

Dominik Csapak d.csapak at proxmox.com
Mon Feb 19 14:59:56 CET 2024


On 2/19/24 14:57, Thomas Lamprecht wrote:
> Am 07/12/2023 um 14:35 schrieb Dominik Csapak:
>> with this the title is now completely visible when the column is
>> expanded, even if it's originally too narrow.
>>
>> Suggested-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
>> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
>> ---
>> changes from v1:
>> * wrap the text in the column, instead of duplicating it in the expanded
>>    body
>>
>> thanks @thomas for the suggestion
>>
>>   www/css/ext6-pve.css        |  4 ++++
>>   www/manager6/ceph/Status.js | 16 ++++++++++++++++
>>   2 files changed, 20 insertions(+)
>>
>>
> 
> applied, with Fiona's T-b and the fix for the eslint warning squashed in,
> thanks!
> 
> btw. I first tried `pve-eslint --fix .` inside the (clean) www/manager6
> folder, but that just resulted in a truncated ceph/Status.js file, so while
> eslint seems to correctly get the fix option and finds the issue, writing
> out the updated code to the file seems broken – can you look into that?
> (no pressure though)

i guess you meant me with that ;)
yes i can look into that




More information about the pve-devel mailing list