Skip to content

Conversation

@sloorush
Copy link
Member

No description provided.

@sloorush
Copy link
Member Author

@cursork, not sure if we would want to make a util function for edge styles to not have to write the same code multiple times.

@sloorush
Copy link
Member Author

sloorush commented Dec 24, 2025

@sloorush sloorush requested a review from cursork December 24, 2025 06:56
@sloorush sloorush marked this pull request as ready for review December 24, 2025 06:56
@cursork
Copy link
Contributor

cursork commented Dec 31, 2025

We can definitely move the function for getting the edge style somewhere shared.

When we're together, we can decide on the best styling. I created a little test using a dfn to swap ⎕WC and eWC and we'll need some styling tweaks. Can be done whilst getting the edge style definition in one place.

Screenshot 2025-12-31 at 12 59 33

Function:

 EdgeStylesXWC;styles;spacing;ypos;nm;F;title

 Called as: xWC←{⍺ ⎕WC ⍵} ⋄ EdgeStylesXWC ⋄ xWC←{⍺ eWC ⍵} ⋄ EdgeStylesXWC

 (F'F',?100)#.xWC'Form'(title'EdgeStyle Demo. xWC=',⎕CR'#.xWC')(50 50)(750 900)('Coord' 'Pixel')

 (F,'.TITLEFNT')#.xWC'Font' 'Segoe UI'('Size' 18)('Weight' 700)
 (F,'.TITLE')#.xWC'Label'title(10 10)(30 880)('FontObj' 'F1.TITLEFNT')
 styles'Ridge' 'Groove' 'Recess' 'Plinth' 'Shadow' 'None'
 spacing ypos105 100

 :For i :In styles
     nm(F,'.E'),(i),'Lbl'  nm #.xWC'Label'((istyles),':')(ypos+10,10+spacing×i-1)(20 90)
     nm(F,'.E'),i  nm #.xWC'Edit' 'Sample Text'(ypos+50,10+spacing×i-1)(25 90)('EdgeStyle'(istyles))
 :EndFor

@sloorush
Copy link
Member Author

sloorush commented Jan 6, 2026

@cursork, sounds good! The demo looks nice. We can check out the styling when we are pairing.

@sloorush
Copy link
Member Author

sloorush commented Jan 8, 2026

Todo:

  • Plinth to be changed to 1px
  • Shadow styling to be made same as ⎕WC
  • We will ignore recess
  • Move all the functions to a standard place

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.

Border is not supported on multiple components Group: Implement EdgeStyle

3 participants